Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Update replication stream docs to better describe role of POSITION #16560

Open
erikjohnston opened this issue Oct 27, 2023 · 1 comment
Open

Update replication stream docs to better describe role of POSITION #16560

erikjohnston opened this issue Oct 27, 2023 · 1 comment
Labels
A-Docs things relating to the documentation A-Workers Problems related to running Synapse in Worker Mode (or replication) T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.

Comments

@erikjohnston
Copy link
Member

c.f. #16555 (comment)
@erikjohnston erikjohnston mentioned this issue Oct 27, 2023
Merged
@DMRobertson
Copy link
Contributor

DMRobertson commented Oct 27, 2023 •
edited
Loading

Examples are the key here, I think.

An incoming POSITION sends two numbers prev < next. (I assume that prev = next or next < prev are invalid?) The worker processing the POSITION has its own position "current". There are then five cases to illustrate:

  • current < prev < next
  • current = prev < next
  • prev < current < next
  • prev < current = next
  • prev < next < current

(The invariant we apparently want is prev <= current <= next, i.e. the middle three bullets.)

@DMRobertson DMRobertson added A-Docs things relating to the documentation A-Workers Problems related to running Synapse in Worker Mode (or replication) T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks. labels Oct 27, 2023
@matrixbot matrixbot mentioned this issue Dec 22, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
A-Docs things relating to the documentation A-Workers Problems related to running Synapse in Worker Mode (or replication) T-Task Refactoring, removal, replacement, enabling or disabling functionality, other engineering tasks.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@erikjohnston @DMRobertson