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

Commit

Permalink
Remove stream token-per-persister removal hack
Browse files Browse the repository at this point in the history
This was an old issue that is now believed to be fixed in synapse.
  • Loading branch information
Fizzadar committed Jan 29, 2024
1 parent 5e7bff6 commit 15f0bf2
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions synapse/handlers/sync.py
Original file line number Diff line number Diff line change
Expand Up @@ -1438,16 +1438,6 @@ async def generate_sync_result(
# Always use the `now_token` in `SyncResultBuilder`
now_token = self.event_sources.get_current_token()

# Beeper change: remove any per-instance stream token positions, this
# happens when one or more event persisters fall behind the others, in
# practice this is very rare and usually only ~1-2 events. By removing
# this we dramatically simplify debugging issues, checking caches and
# generating room previews.
now_token = now_token.copy_and_replace(
StreamKeyType.ROOM,
RoomStreamToken(None, now_token.room_key.stream),
)

if debug_current_token:
logger.info(
"Overriding sync current token for debugging to: %r",
Expand Down

0 comments on commit 15f0bf2

Please sign in to comment.