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.

Sign up for GitHub

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

Jump to bottom

blockchaintest: Optional partial state hash check #975

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

blockchaintest: Optional partial state hash check #975

wants to merge 1 commit into from

Commits on Sep 10, 2024

  1. blockchaintest: Optional partial state hash check 

    This is blockchain tests execution optimization: for listed test names
only check state root hash of first 5 blocks (to detect early problems)
and last 5 blocks (to do the final check of the chain of blocks).

The current implementation of the MPT hash of the state builds
the trie from scratch (no updates to the trie of the previous block).
For the tests will a long chain of blocks the performance degrades
significantly with 99% time spent in the keccak hash function.

This improves testing of EIP-2935
implemented in #953.
    @chfast
    chfast committed Sep 10, 2024
    Configuration menu
    Copy the full SHA
    14ae941 View commit details
    Browse the repository at this point in the history