-
Notifications
You must be signed in to change notification settings - Fork 840
Migrate X-Chain from LevelDB to Firewood and PebbleDB #3732
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
Open
joshua-kim
wants to merge
1
commit into
x-chain-merkle-db-firewood
Choose a base branch
from
x-chain-merkle-db
base: x-chain-merkle-db-firewood
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
eb872ad to
f380ef8
Compare
joshua-kim
commented
Mar 4, 2025
f380ef8 to
411ba8a
Compare
5e741a1 to
409b2fe
Compare
9545412 to
daab96b
Compare
e89db38 to
20ae963
Compare
db6f8b7 to
0505956
Compare
040debc to
de2d05b
Compare
daab96b to
14cb603
Compare
de2d05b to
15ad838
Compare
joshua-kim
commented
Mar 18, 2025
joshua-kim
commented
Mar 18, 2025
joshua-kim
commented
Mar 18, 2025
67bc984 to
44d5295
Compare
33d2d4f to
a8d0c6d
Compare
6468397 to
e2959b4
Compare
This comment has been minimized.
This comment has been minimized.
1f141e7 to
30991e6
Compare
Contributor
Author
|
Need to come up with a way to break up this diff. Currently I think that we could break this into:
|
72f2b25 to
28ce9f4
Compare
35e88ec to
fb34278
Compare
This comment has been minimized.
This comment has been minimized.
fb34278 to
8c16c9a
Compare
Signed-off-by: Joshua Kim <20001595+joshua-kim@users.noreply.github.com>
8c16c9a to
1de8517
Compare
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.
Why this should be merged
Implements a database migration from the legacy database format into a database format that supports a merkle-ized prefix of state
How this works
Changes db schema
How this was tested
Added a migration unit test + tested state migration on Fuji
Need to be documented in RELEASES.md?
Yes