Merge Compatible to develop oct22 #17989
Merged
+794
−152
Conversation
This file contains hidden or 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
If a daemon is stopped at certain points while syncing a converting ledger to a network, the underlying databases can be left in an unexpected state: the num_accounts in the database will be set to a high value (because the daemon added a batch of accounts at high addresses) but not all accounts at lower indices will be present (because the daemon didn't get to filling them in yet). This will cause functions like iteri and get_at_index_exn to throw unexpected exceptions when they get to addresses that are missing accounts. A couple of new ledger functions have been added to account for this possibility, which are used in an updated ledger database sync check implementation. A few related unit tests have been added to the converting merkle tree, one of which does not pass without these changes being in place.
The ~here is set to the point of usage of Option.value_exn itself to conform with our code base's current practices, but we might want to have it passed in from the caller of these functions.
track new o1Labs OPAM repo. 1
…n-nix Bump RocksDB to 10.5.1 for non-nix builds
…k-more-defensive Make converting ledger sync check more defensive
[CI] Support archs argument in aptly
…dups [CI] Fix packages name conflict between mina-devnet and mina-devnet-lightnet
…missing_bl_auditor Handling archive from network which were derived from fork
[CMP] port merge 3.3.0 to compatible
Rename Lucy Test: hard_fork -> post_hard_fork
…bian-as-multi Mark Mina Artifact Debian as Multi
implement RocksDB compatibility test in Python
… of the literal 0. Query on Archive DB to figure out FK ID to field elemnt 0.
…p-state-migrate-v2 Fix Archive Migration Script for Berkeley -> Mesa v2
Remove README-spacetime.md
glyh
changed the title
|
!ci-build-me |
dkijania
approved these changes
Oct 22, 2025
|
!ci-bypass-changelog |
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.
This should fix failing rosetta devnet connectivity test on devleop if there's any.
This is an automatic merge.