Merge pull request #6255 from matrix-org/erikj/stats_config

Remove repeated calls to config.stats_enabled.

changelog.d/6255.misc

@@ -0,0 +1 @@
+Small performance improvement by removing repeated config lookups in room stats calculation.

synapse/handlers/stats.py

@@ -45,6 +45,8 @@ def __init__(self, hs):
         self.is_mine_id = hs.is_mine_id
         self.stats_bucket_size = hs.config.stats_bucket_size
 
+        self.stats_enabled = hs.config.stats_enabled
+
         # The current position in the current_state_delta stream
         self.pos = None
 
@@ -61,7 +63,7 @@ def __init__(self, hs):
     def notify_new_event(self):
         """Called when there may be more deltas to process
         """
-        if not self.hs.config.stats_enabled or self._is_processing:
+        if not self.stats_enabled or self._is_processing:
             return
 
         self._is_processing = True

tests/handlers/test_stats.py

@@ -607,6 +607,7 @@ def test_initial_background_update(self):
         """
 
         self.hs.config.stats_enabled = False
+        self.handler.stats_enabled = False
 
         u1 = self.register_user("u1", "pass")
         u1token = self.login("u1", "pass")
@@ -618,6 +619,7 @@ def test_initial_background_update(self):
         self.assertIsNone(self._get_current_stats("user", u1))
 
         self.hs.config.stats_enabled = True
+        self.handler.stats_enabled = True
 
         self._perform_background_initial_update()