summary refs log tree commit diff
path: root/synapse/util/caches/lrucache.py
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2018-03-15 15:40:13 +0000
committerErik Johnston <erik@matrix.org>2018-03-15 15:46:54 +0000
commit7c7706f42b56dd61f5eb17679aa12247f7058ed5 (patch)
tree5d45938bcff322833a04e8f072507f7fcdb71b81 /synapse/util/caches/lrucache.py
parentMerge pull request #3003 from matrix-org/rav/fix_contributing (diff)
downloadsynapse-7c7706f42b56dd61f5eb17679aa12247f7058ed5.tar.xz
Fix bug where state cache used lots of memory
The state cache bases its size on the sum of the size of entries. The
size of the entry is calculated once on insertion, so it is important
that the size of entries does not change.

The DictionaryCache modified the entries size, which caused the state
cache to incorrectly think it was smaller than it actually was.
Diffstat (limited to '')
-rw-r--r--synapse/util/caches/lrucache.py8
1 files changed, 4 insertions, 4 deletions
diff --git a/synapse/util/caches/lrucache.py b/synapse/util/caches/lrucache.py
index f088dd430e..a4bf8fa6ae 100644
--- a/synapse/util/caches/lrucache.py
+++ b/synapse/util/caches/lrucache.py
@@ -154,14 +154,14 @@ class LruCache(object):
         def cache_set(key, value, callbacks=[]):
             node = cache.get(key, None)
             if node is not None:
-                if value != node.value:
+                if node.callbacks and value != node.value:
                     for cb in node.callbacks:
                         cb()
                     node.callbacks.clear()
 
-                    if size_callback:
-                        cached_cache_len[0] -= size_callback(node.value)
-                        cached_cache_len[0] += size_callback(value)
+                if size_callback:
+                    cached_cache_len[0] -= size_callback(node.value)
+                    cached_cache_len[0] += size_callback(value)
 
                 node.callbacks.update(callbacks)