Fix bug that could cause a /sync to tightloop with sqlite after res…

…tart (#16540) This could happen if the last rows in the account data stream were inserted into `account_data`. After a restart the max account ID would be calculated without looking at the `account_data` table, and so have an old ID.
matrix-org · Oct 23, 2023 · 3bc23cc · 3bc23cc
1 parent 3bcb6a0
commit 3bc23cc
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/changelog.d/16540.bugfix b/changelog.d/16540.bugfix
@@ -0,0 +1 @@
+Fix long-standing bug where `/sync` could tightloop after restart when using SQLite.
diff --git a/synapse/storage/databases/main/account_data.py b/synapse/storage/databases/main/account_data.py
@@ -94,7 +94,10 @@ def __init__(
                 hs.get_replication_notifier(),
                 "room_account_data",
                 "stream_id",
-                extra_tables=[("room_tags_revisions", "stream_id")],
+                extra_tables=[
+                    ("account_data", "stream_id"),
+                    ("room_tags_revisions", "stream_id"),
+                ],
                 is_writer=self._instance_name in hs.config.worker.writers.account_data,
             )
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Fix long-standing bug where `/sync` could tightloop after restart when using SQLite.