diff options
author | Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> | 2021-02-19 11:37:29 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-02-19 11:37:29 +0000 |
commit | 8bcfc2eaad66aa6e92a745d226fee4a81a921541 (patch) | |
tree | 3d56cef2ce4f36d35f7e503ffd30743841fde3b0 /changelog.d/9402.bugfix | |
parent | Add a config option to prioritise local users in user directory search result... (diff) | |
download | synapse-8bcfc2eaad66aa6e92a745d226fee4a81a921541.tar.xz |
Be smarter about which hosts to send presence to when processing room joins (#9402)
This PR attempts to eliminate unnecessary presence sending work when your local server joins a room, or when a remote server joins a room your server is participating in by processing state deltas in chunks rather than individually. --- When your server joins a room for the first time, it requests the historical state as well. This chunk of new state is passed to the presence handler which, after filtering that state down to only membership joins, will send presence updates to homeservers for each join processed. It turns out that we were being a bit naive and processing each event individually, and sending out presence updates for every one of those joins. Even if many different joins were users on the same server (hello IRC bridges), we'd send presence to that same homeserver for every remote user join we saw. This PR attempts to deduplicate all of that by processing the entire batch of state deltas at once, instead of only doing each join individually. We process the joins and note down which servers need which presence: * If it was a local user join, send that user's latest presence to all servers in the room * If it was a remote user join, send the presence for all local users in the room to that homeserver We deduplicate by inserting all of those pending updates into a dictionary of the form: ``` { server_name1: {presence_update1, ...}, server_name2: {presence_update1, presence_update2, ...} } ``` Only after building this dict do we then start sending out presence updates.
Diffstat (limited to 'changelog.d/9402.bugfix')
-rw-r--r-- | changelog.d/9402.bugfix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/9402.bugfix b/changelog.d/9402.bugfix new file mode 100644 index 0000000000..7729225ba2 --- /dev/null +++ b/changelog.d/9402.bugfix @@ -0,0 +1 @@ +Fix a bug where a lot of unnecessary presence updates were sent when joining a room. |