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

fix: use prefix on state trie keys #200

Merged
merged 7 commits into from
Jan 2, 2024
Merged

fix: use prefix on state trie keys #200

merged 7 commits into from
Jan 2, 2024

Conversation

kroggen
Copy link
Member

@kroggen kroggen commented May 9, 2023

No description provided.

@kroggen kroggen added the bug fix This fixes a bug label Sep 28, 2023
@rabbitprincess rabbitprincess mentioned this pull request Oct 16, 2023
@rabbitprincess
Copy link
Member

rabbitprincess commented Oct 30, 2023

it need to pass sync test from new empty db. @kslee8282

@rabbitprincess rabbitprincess added the check hardfork Run sync test with the PR label Oct 30, 2023
@kslee8282
Copy link
Member

it need to pass sync test from new empty db. @kslee8282

Oct 30 04:04:01.342 INF ../go/aergo/cmd/aergosvr/aergosvr.go:100 > AERGO SVR STARTED branch=fix/state-trie-keys module=asvr revision=d17db23a

start sync mainnet

@kslee8282
Copy link
Member

testnet sync (11821274 / 111859080)
mainnet sync (81317246 / 142831796)

@kslee8282
Copy link
Member

sync success

@kslee8282 kslee8282 added the pass sync test The PR can be merged label Nov 6, 2023
@rabbitprincess
Copy link
Member

rabbitprincess commented Nov 6, 2023

it can be merged after all node ready to migration on next version.

@rabbitprincess rabbitprincess self-requested a review January 2, 2024 07:29
Copy link
Member

@rabbitprincess rabbitprincess left a comment

Choose a reason for hiding this comment

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

db migration is ready.

@kroggen kroggen merged commit 55e3347 into develop Jan 2, 2024
3 checks passed
@kroggen kroggen deleted the fix/state-trie-keys branch January 2, 2024 17:34
This was referenced Jan 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug fix This fixes a bug check hardfork Run sync test with the PR pass sync test The PR can be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants