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

More efficiently handle no-op POSITION #16640

Merged
merged 5 commits into from
Nov 16, 2023
Merged

Conversation

erikjohnston
Copy link
Member

We may receive POSITION commands where we already know that worker has advanced past that position, so there is no point in handling it.

We may receive `POSITION` commands where we already know that worker has
advanced past that position, so there is no point in handling it.
@erikjohnston erikjohnston marked this pull request as ready for review November 15, 2023 16:55
@erikjohnston erikjohnston requested a review from a team as a code owner November 15, 2023 16:55
synapse/replication/tcp/handler.py Outdated Show resolved Hide resolved
synapse/replication/tcp/handler.py Outdated Show resolved Hide resolved
synapse/replication/tcp/streams/_base.py Outdated Show resolved Hide resolved
synapse/replication/tcp/streams/_base.py Outdated Show resolved Hide resolved
@erikjohnston erikjohnston merged commit 898655f into develop Nov 16, 2023
38 of 41 checks passed
@erikjohnston erikjohnston deleted the erikj/discard_positions branch November 16, 2023 12:32
erikjohnston added a commit that referenced this pull request Nov 17, 2023
erikjohnston added a commit that referenced this pull request Nov 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants