diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2022-01-18 13:06:04 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-01-18 13:06:04 +0000 |
| commit | 251b5567ecc8fb3d6debaa3f77f6ec2620877d36 (patch) | |
| tree | e3621904f7c20ba45688163488cd17c8922b8e1d /synapse/notifier.py | |
| parent | Merge branch 'master' into develop (diff) | |
| download | synapse-251b5567ecc8fb3d6debaa3f77f6ec2620877d36.tar.xz | |
Remove `log_function` and its uses (#11761)
I've never found this terribly useful. I think it was added in the early days of Synapse, without much thought as to what would actually be useful to log, and has just been cargo-culted ever since. Rather, it tends to clutter up debug logs with useless information.
Diffstat (limited to 'synapse/notifier.py')
| -rw-r--r-- | synapse/notifier.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/synapse/notifier.py b/synapse/notifier.py index 41fd94d772..632b2245ef 100644 --- a/synapse/notifier.py +++ b/synapse/notifier.py @@ -40,7 +40,6 @@ from synapse.handlers.presence import format_user_presence_state from synapse.logging import issue9533_logger from synapse.logging.context import PreserveLoggingContext from synapse.logging.opentracing import log_kv, start_active_span -from synapse.logging.utils import log_function from synapse.metrics import LaterGauge from synapse.streams.config import PaginationConfig from synapse.types import ( @@ -686,7 +685,6 @@ class Notifier: else: return False - @log_function def remove_expired_streams(self) -> None: time_now_ms = self.clock.time_msec() expired_streams = [] @@ -700,7 +698,6 @@ class Notifier: for expired_stream in expired_streams: expired_stream.remove(self) - @log_function def _register_with_keys(self, user_stream: _NotifierUserStream): self.user_to_user_stream[user_stream.user_id] = user_stream |