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/federation/persistence.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/federation/persistence.py')
| -rw-r--r-- | synapse/federation/persistence.py | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/synapse/federation/persistence.py b/synapse/federation/persistence.py index 523ab1c51e..60e2e6cf01 100644 --- a/synapse/federation/persistence.py +++ b/synapse/federation/persistence.py @@ -23,7 +23,6 @@ import logging from typing import Optional, Tuple from synapse.federation.units import Transaction -from synapse.logging.utils import log_function from synapse.storage.databases.main import DataStore from synapse.types import JsonDict @@ -36,7 +35,6 @@ class TransactionActions: def __init__(self, datastore: DataStore): self.store = datastore - @log_function async def have_responded( self, origin: str, transaction: Transaction ) -> Optional[Tuple[int, JsonDict]]: @@ -53,7 +51,6 @@ class TransactionActions: return await self.store.get_received_txn_response(transaction_id, origin) - @log_function async def set_response( self, origin: str, transaction: Transaction, code: int, response: JsonDict ) -> None: |