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

Include additional information to output batches MDBX browser tool #1047

Merged
merged 7 commits into from
Aug 30, 2024

Conversation

Stefan-Ethernal
Copy link
Collaborator

@Stefan-Ethernal Stefan-Ethernal commented Aug 28, 2024

This PR aligns (as much as possible, since it is an offline tool) output of output-batches CLI command in the MDBX DB browser tool with the zkevm_getBatchByNumber.

It also implements new command called output-batch-affiliation, that prints the batch numbers, for the provided block numbers.

@cla-bot cla-bot bot added the cla-signed label Aug 28, 2024
@Stefan-Ethernal Stefan-Ethernal self-assigned this Aug 28, 2024
@Stefan-Ethernal Stefan-Ethernal force-pushed the fix/output-batches-by-number-mdbx-db-tool branch from 88dd4fa to 3851b84 Compare August 28, 2024 17:31
revitteth
revitteth previously approved these changes Aug 29, 2024
@Stefan-Ethernal Stefan-Ethernal enabled auto-merge (squash) August 30, 2024 07:13
@Stefan-Ethernal Stefan-Ethernal force-pushed the fix/output-batches-by-number-mdbx-db-tool branch from e704269 to cb1e789 Compare August 30, 2024 09:34
@Stefan-Ethernal Stefan-Ethernal force-pushed the fix/output-batches-by-number-mdbx-db-tool branch from cb1e789 to 586c8cd Compare August 30, 2024 09:42
@Stefan-Ethernal Stefan-Ethernal force-pushed the fix/output-batches-by-number-mdbx-db-tool branch from 586c8cd to 862ae6b Compare August 30, 2024 09:43
Copy link

@Stefan-Ethernal Stefan-Ethernal merged commit 77d07b8 into zkevm Aug 30, 2024
10 checks passed
@Stefan-Ethernal Stefan-Ethernal deleted the fix/output-batches-by-number-mdbx-db-tool branch August 30, 2024 10:28
Stefan-Ethernal added a commit that referenced this pull request Sep 20, 2024
…1047)

* Populate additional details in the batch (WIP)

* Use the const for the fork id identifier

* GetBatchDataByNumbers add error handling and remove transactions collection

* Populate additional fields for the batches

* Implement output-batch-affiliation command

* Fix tests

* Fix DbDataRetriever UT
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants