diff options
author | Richard van der Hoff <richard@matrix.org> | 2018-09-26 19:17:36 +0100 |
---|---|---|
committer | Richard van der Hoff <richard@matrix.org> | 2018-09-27 11:31:51 +0100 |
commit | 28223841e05a77a44ec2c0b29d1e930c68974913 (patch) | |
tree | 9c80e36e880af1d38d368503dae142bac828438f /synapse/handlers/federation.py | |
parent | changelog (diff) | |
download | synapse-28223841e05a77a44ec2c0b29d1e930c68974913.tar.xz |
more comments
Diffstat (limited to 'synapse/handlers/federation.py')
-rw-r--r-- | synapse/handlers/federation.py | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/synapse/handlers/federation.py b/synapse/handlers/federation.py index 6793f9b6c6..38bebbf598 100644 --- a/synapse/handlers/federation.py +++ b/synapse/handlers/federation.py @@ -341,10 +341,9 @@ class FederationHandler(BaseHandler): ) with logcontext.nested_logging_context(p): - # XXX if any of the missing prevs share missing state or auth - # events, we'll end up requesting those missing events for - # *each* missing prev, contributing to the hammering of /event - # as per https://github.com/matrix-org/synapse/issues/2164. + # note that if any of the missing prevs share missing state or + # auth events, the requests to fetch those events are deduped + # by the get_pdu_cache in federation_client. remote_state, got_auth_chain = ( yield self.federation_client.get_state_for_room( origin, room_id, p, |