summary refs log tree commit diff
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2017-07-14 16:11:26 +0100
committerErik Johnston <erik@matrix.org>2017-07-14 16:11:26 +0100
commitbfde0760224c09a5e6327d4ae4181ecb10ccfc2e (patch)
treee39c1b6f922aaf5434a1007f88f5a480d32d891d
parentMerge pull request #2366 from matrix-org/erikj/push_metrics (diff)
downloadsynapse-bfde0760224c09a5e6327d4ae4181ecb10ccfc2e.tar.xz
Increase cache hit ratio for push
We don't update the cache in all code paths, which causes subsequent
calls to miss the cache
-rw-r--r--synapse/push/bulk_push_rule_evaluator.py8
1 files changed, 8 insertions, 0 deletions
diff --git a/synapse/push/bulk_push_rule_evaluator.py b/synapse/push/bulk_push_rule_evaluator.py
index 9134969553..b0d64aa6c4 100644
--- a/synapse/push/bulk_push_rule_evaluator.py
+++ b/synapse/push/bulk_push_rule_evaluator.py
@@ -316,6 +316,14 @@ class RulesForRoom(object):
                 yield self._update_rules_with_member_event_ids(
                     ret_rules_by_user, missing_member_event_ids, state_group, event
                 )
+            else:
+                # The push rules didn't change but lets update the cache anyway
+                self.update_cache(
+                    self.sequence,
+                    members={},  # There were no membership changes
+                    rules_by_user=ret_rules_by_user,
+                    state_group=state_group
+                )
 
         if logger.isEnabledFor(logging.DEBUG):
             logger.debug(