summary refs log tree commit diff
path: root/scripts-dev/release.py
diff options
context:
space:
mode:
authorEric Eastwood <erice@element.io>2021-11-02 17:27:04 -0500
committerEric Eastwood <erice@element.io>2021-11-02 17:27:04 -0500
commit321f9ea68b4b87da91dc96c22f7c466a91d46c68 (patch)
treea7e546ea02f3c81db6f1117bbfa3a8ff2ab2f0c2 /scripts-dev/release.py
parentFix lints (diff)
downloadsynapse-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/release.py')
0 files changed, 0 insertions, 0 deletions