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

Db migration app from mainnet to berkeley #12906

Merged
merged 24 commits into from
Nov 1, 2023

Conversation

psteckler
Copy link
Member

@psteckler psteckler commented Mar 23, 2023

App to migrate mainnet archive db data to berkeley schema. Ledger hashes and snarked ledger hashes are left intact. The implementation converts each block into the "extensional" format, and imports that into the new db.

Notes:

  • the transaction hashes are passed through unaltered
  • the app will resume where it left off, allowing incremental migration
  • the accounts_accessed table is not populated, that requires the ledger contents; a modified replayer will do that in a separate migration step

Part of #12676.

Part of #14313.

Closes #10945.

@psteckler psteckler requested review from a team as code owners March 23, 2023 18:55
@psteckler
Copy link
Member Author

!ci-build-me

@psteckler
Copy link
Member Author

!ci-build-me

@psteckler
Copy link
Member Author

!ci-build-me

@psteckler
Copy link
Member Author

!ci-build-me

|sql} )
(Mina_caqti.insert_into_cols ~returning:"id" ~table_name
~tannot:(function
| "sub_window_densities" ->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would this change affect the archiver on berkeley branch?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean the berkeley testnet (this PR is against the berkeley branch)?

This fix affects only archiving extensional blocks, so it doesn't affect the processor running on the testnet.

@psteckler
Copy link
Member Author

!ci-build-me

@deepthiskumar
Copy link
Member

!approved-for-mainnet

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants