diff options
| author | David Robertson <davidr@element.io> | 2021-09-22 10:59:52 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-09-22 10:59:52 +0100 |
| commit | a2d7195e0111ee6b2fedaabb0f02cfae648cd347 (patch) | |
| tree | 73c583a70aa2dc34a87622fbc678b65a755f849e /synapse/util/caches/lrucache.py | |
| parent | Rename MSC2716 things from `chunk` to `batch` to match `/batch_send` endpoint... (diff) | |
| download | synapse-a2d7195e0111ee6b2fedaabb0f02cfae648cd347.tar.xz | |
Track why we're evicting from caches (#10829)
So we can see distinguish between "evicting because the cache is too big" and "evicting because the cache entries haven't been recently used".
Diffstat (limited to 'synapse/util/caches/lrucache.py')
| -rw-r--r-- | synapse/util/caches/lrucache.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/synapse/util/caches/lrucache.py b/synapse/util/caches/lrucache.py index 39dce9dd41..ea6e8dc8d1 100644 --- a/synapse/util/caches/lrucache.py +++ b/synapse/util/caches/lrucache.py @@ -40,7 +40,7 @@ from twisted.internet.interfaces import IReactorTime from synapse.config import cache as cache_config from synapse.metrics.background_process_metrics import wrap_as_background_process from synapse.util import Clock, caches -from synapse.util.caches import CacheMetric, register_cache +from synapse.util.caches import CacheMetric, EvictionReason, register_cache from synapse.util.caches.treecache import TreeCache, iterate_tree_cache_entry from synapse.util.linked_list import ListNode @@ -403,7 +403,7 @@ class LruCache(Generic[KT, VT]): evicted_len = delete_node(node) cache.pop(node.key, None) if metrics: - metrics.inc_evictions(evicted_len) + metrics.inc_evictions(EvictionReason.size, evicted_len) def synchronized(f: FT) -> FT: @wraps(f) |