aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKim Alvefur <zash@zash.se>2015-12-06 02:09:52 +0100
committerKim Alvefur <zash@zash.se>2015-12-06 02:09:52 +0100
commit8920afaf2fb1d0b724f9528d009c0bc226c3adbc (patch)
treebad31cc308c3fe56803d11b69affdbd9069f2d99
parent4b1ba49a85adf72d3cbdb2b8cb83e52b1ed4e9e6 (diff)
downloadprosody-8920afaf2fb1d0b724f9528d009c0bc226c3adbc.tar.gz
prosody-8920afaf2fb1d0b724f9528d009c0bc226c3adbc.zip
mod_blocklist: Expand comments on caching of blocklists
-rw-r--r--plugins/mod_blocklist.lua12
1 files changed, 9 insertions, 3 deletions
diff --git a/plugins/mod_blocklist.lua b/plugins/mod_blocklist.lua
index add7abb3..a38d2a0f 100644
--- a/plugins/mod_blocklist.lua
+++ b/plugins/mod_blocklist.lua
@@ -19,11 +19,17 @@ local jid_split = require"util.jid".split;
local storage = module:open_store();
local sessions = prosody.hosts[module.host].sessions;
--- Cache of blocklists by username may randomly expire at any time
+-- First level cache of blocklists by username.
+-- Weak table so may randomly expire at any time.
local cache = setmetatable({}, { __mode = "v" });
--- Second level of caching, keeps a fixed number of items,
--- also anchors items in the above cache
+-- Second level of caching, keeps a fixed number of items, also anchors
+-- items in the above cache.
+--
+-- The size of this affects how often we will need to load a blocklist from
+-- disk, which we want to avoid during routing. On the other hand, we don't
+-- want to use too much memory either, so this can be tuned by advanced
+-- users. TODO use science to figure out a better default, 64 is just a guess.
local cache_size = module:get_option_number("blocklist_cache_size", 64);
local cache2 = require"util.cache".new(cache_size);