Skip to content
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

[Epic]: migration of the new store module #15769

Closed
cool-develope opened this issue Apr 10, 2023 · 3 comments
Closed

[Epic]: migration of the new store module #15769

cool-develope opened this issue Apr 10, 2023 · 3 comments

Comments

@cool-develope
Copy link
Contributor

Since we are using a new node key format (cosmos/iavl#676), we need to migrate the original iavl store to the new version.
The migration will proceed in two main stages. Let's assume the current store keeps fromHeight to toHeight of blocks.

  • it will create the snapshot for the fromHeight through exporting in the original version and then import that snapshot into the new version.
  • it will iterate ChangeSets for fromHeight+1 to toHeight using TraverseStateChanges in the original version and then save those ChangeSets using SaveChangeSet in the new version

ref: #15568, cosmos/iavl#675, cosmos/iavl#676

@cool-develope
Copy link
Contributor Author

@tac0turtle ☝️

@tac0turtle
Copy link
Member

will this all be handled from within IAVL? i think having the store pacakage handle the transition may lead to unforeseen complexity

@github-project-automation github-project-automation bot moved this to 📝 Todo in Cosmos-SDK Apr 11, 2023
@tac0turtle tac0turtle moved this from 📝 Todo to 💪 In Progress in Cosmos-SDK Apr 17, 2023
@tac0turtle
Copy link
Member

closed via #15568

@github-project-automation github-project-automation bot moved this from 💪 In Progress to 👏 Done in Cosmos-SDK Jun 7, 2023
@tac0turtle tac0turtle removed this from Cosmos-SDK Jul 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants