This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Port script cannot be run twice on the same DB #8619
Labels
S-Minor
Blocks non-critical functionality, workarounds exist.
T-Defect
Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
X-Regression
Something broke which worked on a previous release
z-p2
(Deprecated Label)
Comments
erikjohnston
added a commit
that referenced
this issue
Nov 13, 2020
If the script fails (or is CTRL-C'ed) between porting some of the events table and copying of the sequences then the port script will immediately die if run again due to the postgres DB having inconsistencies between sequences and tables. Fixes #8619
erikjohnston
added a commit
that referenced
this issue
Nov 13, 2020
If the script fails (or is CTRL-C'ed) between porting some of the events table and copying of the sequences then the port script will immediately die if run again due to the postgres DB having inconsistencies between sequences and tables. The fix is to move the porting of sequences to before porting the tables, so that there is never a period where the Postgres DB is inconsistent. To do that we need to change how we port the sequences so that it calculates the values from the SQLite DB rather than the Postgres DB. Fixes #8619
this seems to have regressed:
|
It's actually worse than that - Synapse won't start when pointed to the ported database. This isn't really a regression of the original bug, so I'm going to close this in favour of #9979. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
S-Minor
Blocks non-critical functionality, workarounds exist.
T-Defect
Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
X-Regression
Something broke which worked on a previous release
z-p2
(Deprecated Label)
The port script should be able to run multiple times on the same DB, to allow incremental porting.
This fails with:
The text was updated successfully, but these errors were encountered: