diff options
| author | Erik Johnston <erikj@matrix.org> | 2023-10-23 16:57:30 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-23 16:57:30 +0100 |
| commit | 8f35f8148e1a7ce3ac249e2d2052854409f2c0d6 (patch) | |
| tree | 72eb736c64970a416e07c123a4b350614dc1518d /changelog.d/16473.bugfix | |
| parent | Fix bug that could cause a `/sync` to tightloop with sqlite after restart (#1... (diff) | |
| download | synapse-8f35f8148e1a7ce3ac249e2d2052854409f2c0d6.tar.xz | |
Fix bug where a new writer advances their token too quickly (#16473)
* Fix bug where a new writer advances their token too quickly When starting a new writer (for e.g. persisting events), the `MultiWriterIdGenerator` doesn't have a minimum token for it as there are no rows matching that new writer in the DB. This results in the the first stream ID it acquired being announced as persisted *before* it actually finishes persisting, if another writer gets and persists a subsequent stream ID. This is due to the logic of setting the minimum persisted position to the minimum known position of across all writers, and the new writer starts off not being considered. * Fix sending out POSITIONs when our token advances without update Broke in #14820 * For replication HTTP requests, only wait for minimal position
Diffstat (limited to 'changelog.d/16473.bugfix')
| -rw-r--r-- | changelog.d/16473.bugfix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/16473.bugfix b/changelog.d/16473.bugfix new file mode 100644 index 0000000000..4f4a0380cd --- /dev/null +++ b/changelog.d/16473.bugfix @@ -0,0 +1 @@ +Fix a long-standing, exceedingly rare edge case where the first event persisted by a new event persister worker might not be sent down `/sync`. |