Never throw VersionError #1880
Merged
Never throw VersionError #1880
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Collaborator
dfahlander
commented
Jan 22, 2024
dfahlander
commented
- Allow schema changes without version bump
- Allow downgrades
- Only require version bump when content upgrades are needed
…as long as they have a compatible schema. A compatible schema does not need to have the same tables or indexes. They might differ quite a lot. * Attaching an upgrader to one of them would mean that the upgrader runs even though the first one didn't request it. Therefore, this scenario should be avoided when having upgraders that modify content between versions. * Schemas are not compatible if they index the same property or properties using different flags (unique or multiEntry) * Schemas are not compatible if they have different primary keys However, in most scenarios where we would fail opening one of the DBs, we will now succeed and be able to access the same data.
(Not just 10) When native verno is incremented 10 times to the next 10 multiple, create a table $meta that contains the virtual version. Maintain that table and delete it when no longer needed.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.