diff options
author | Eric Eastwood <erice@element.io> | 2021-11-02 17:27:04 -0500 |
---|---|---|
committer | Eric Eastwood <erice@element.io> | 2021-11-02 17:27:04 -0500 |
commit | 321f9ea68b4b87da91dc96c22f7c466a91d46c68 (patch) | |
tree | a7e546ea02f3c81db6f1117bbfa3a8ff2ab2f0c2 /scripts-dev/mypy_synapse_plugin.py | |
parent | Fix lints (diff) | |
download | synapse-321f9ea68b4b87da91dc96c22f7c466a91d46c68.tar.xz |
Move back to the old get_backfill_events and simplify backfill.
We now rely on the marker events to backfill the base insertion event which puts it as a insertion event extremity. This functionality was already in place (see `handle_marker_event`) and was an easy transition. This way, remote federated homeserver will have the insertion extremity to ask about in backfill and goes down the historical branch no problem because of the depth order and the rest of the DAG navigation happens as normal. Yay simplification! The key breakthrough was discussing all the ways we can find connected insertion events. https://docs.google.com/document/d/1KCEmpnGr4J-I8EeaVQ8QJZKBDu53ViI7V62y5BzfXr0/edit#bookmark=id.1hbt9acs963h The three options we came up were: - Find by insertion event prev_events (this is what we were doing before) - Find connected insertion events by depth - Find connected insertion events by the marker event - This made the most sense since we already backfill the insertion event when a marker event is processed (see `handle_marker_event`). - Gets rid of the extra insertion event lookup in backfill because we know it's already backfilled from the marker processing. - And gets rid of the extra federated lookup we added in this to PR to ask whether the homeserver requesting backfill already has the insertion event (deciding whether we fork to the history branch before we go down the "live" DAG)
Diffstat (limited to 'scripts-dev/mypy_synapse_plugin.py')
0 files changed, 0 insertions, 0 deletions