-
Notifications
You must be signed in to change notification settings - Fork 211
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[ADP-3212] DB migration of delegation store to support voting state #4428
Conversation
c4b3770
to
9ba8744
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! 😊 A couple of suggestions.
One suggestion up for debate: I think it makes sense to rename "active" to "delegating" in order to better disambiguate it from the new "voting" status. 🤔 This would require a migration of the values in the existing table, but with the new migration scheme, I think that we can afford that.
lib/wallet/src/Cardano/Wallet/DB/Store/Delegations/Migrations/Conway/Migration.hs
Outdated
Show resolved
Hide resolved
lib/wallet/src/Cardano/Wallet/DB/Store/Delegations/Migrations/Lean/Migration.hs
Outdated
Show resolved
Hide resolved
5f1bc71
to
b3fc551
Compare
105430b
to
02a631e
Compare
02a631e
to
92b1cc1
Compare
92b1cc1
to
dcca782
Compare
…e migration result
2f84117
to
fe3af5f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent, thank you for the unit test! 😊 Ready to go in my opinion.
- V2: only contain the historical schema
- V3: contains the migration from V2 to V3 schema and the V3 schema itself (Lean introduction)
- V5: contains the migration from V3 to V5 (Conway changes)
- V5 schema takes place of V3 schema at top level
ADP-3212