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

feat(stages): merkle execute stage progress #2866

Merged
merged 3 commits into from
May 29, 2023

Conversation

shekhirin
Copy link
Collaborator

@shekhirin shekhirin commented May 26, 2023

Report progress for MerkleExecute stage based on the number of hashed accounts and storage slots processed vs total in database. For example, this is Sepolia Merkle stage execution after being dropped manually using CLI:

image

This PR doesn't have any breaking DB changes, as we don't need to introduce any additional checkpoint models.


Further work is to calculate a similar progress for MerkleUnwind, but currently not clear how to report it in metrics, so we could see one common Merkle stage chart going up when executing, and down when unwinding.

@shekhirin shekhirin force-pushed the alexey/merkle-progress branch 3 times, most recently from 1845914 to 4cb1e26 Compare May 26, 2023 14:19
@shekhirin shekhirin changed the title feat(stages): merkle stage progress feat(stages): merkle execute stage progress May 29, 2023
@shekhirin shekhirin marked this pull request as ready for review May 29, 2023 09:54
@shekhirin shekhirin added C-enhancement New feature or request A-staged-sync Related to staged sync (pipelines and stages) A-observability Related to tracing, metrics, logs and other observability tools labels May 29, 2023
@onbjerg onbjerg merged commit 05e3a13 into feat/checkpoints May 29, 2023
@onbjerg onbjerg deleted the alexey/merkle-progress branch May 29, 2023 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-observability Related to tracing, metrics, logs and other observability tools A-staged-sync Related to staged sync (pipelines and stages) C-enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants