This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
simplify federation stream traffic #7549
Labels
A-Workers
Problems related to running Synapse in Worker Mode (or replication)
O-Uncommon
Most users are unlikely to come across this or unexpected workflow
S-Tolerable
Minor significance, cosmetic issues, low or no impact to users.
T-Task
Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Comments
DMRobertson
added
T-Task
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The
federationreplication stream has four types of row:edusis completely unused afaict, andkeyed_edusis only used for typing updates. This is somewhat important, as the federation stream is not reliable (we can drop rows during restarts). Additionally, since typing notifications are typically sent to a lot of destinations, there is a lot of duplication.Could/should we replace
keyed_edusandeduswith a singletypingrow type, and while we're at it remove a bunch of dead code inFederationSender/FederationRemoteSendQueue/PerDestinationQueue(egPerDestinationQueue._pending_edusis never used)?The text was updated successfully, but these errors were encountered: