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

Blocks backfilling option - 2 ProposerVM #2241

Closed
wants to merge 51 commits into from

Conversation

abi87
Copy link
Contributor

@abi87 abi87 commented Oct 30, 2023

Why this should be merged

Node that state synced may need to index all blocks and transactions below state summary height. This PR introduced the possibility to do so.
The PR is somewhat large in LOC,but 1100 of these LOC are UTs. ProposerVM has pretty lengthly UTs setup.

How this works

Backfilling is split in the following PRs

How this was tested

extra UTs + successful mainnet sync with #2273 code

@abi87 abi87 changed the title State sync - Blocks backfilling option - ProposerVM #2240 State sync - Blocks backfilling option - ProposerVM Oct 30, 2023
@abi87 abi87 changed the title State sync - Blocks backfilling option - ProposerVM Blocks backfilling option - 2 ProposerVM Oct 30, 2023
@abi87 abi87 force-pushed the backfill_blocks_proposerVM branch from 5659b92 to 2550a87 Compare November 2, 2023 11:35
@abi87 abi87 force-pushed the backfill_blocks_proposerVM branch from 5e5d81f to 4173d31 Compare November 2, 2023 15:15
@abi87 abi87 marked this pull request as ready for review November 2, 2023 19:06
@abi87 abi87 self-assigned this Nov 2, 2023
@abi87 abi87 force-pushed the backfill_blocks_proposerVM branch from 8b643e7 to 4ac552b Compare November 3, 2023 19:55
@abi87 abi87 mentioned this pull request Nov 9, 2023
5 tasks
@abi87 abi87 linked an issue Nov 20, 2023 that may be closed by this pull request
Copy link

github-actions bot commented Feb 4, 2024

This PR has become stale because it has been open for 30 days with no activity. Adding the lifecycle/frozen label will cause this PR to ignore lifecycle events.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Status: In Review 👀
Development

Successfully merging this pull request may close these issues.

State sync block backfilling
2 participants