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

triedb/pathdb: introduce lookup structure to optimize state access #30971

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

rjl493456442
Copy link
Member

@rjl493456442 rjl493456442 commented Dec 30, 2024

This pull request introduces a mechanism to enhance state lookup efficiency in the pathdb by maintaining a lookup structure that bypasses unnecessary diff layer iterations.

Performance data is provided below.

@rjl493456442
Copy link
Member Author

rjl493456442 commented Dec 30, 2024

With this PR (on top of #30661), the block execution performance matches the current master branch.

The state access (storage, account) is slightly faster. The triedb commit is about 2ms slower due to the lookup overhead.

Bench07: PR
Bench08: Master

截屏2024-12-30 15 54 57 截屏2024-12-30 15 56 55 截屏2024-12-30 16 03 34

Without this PR, #30661 vs master, the state access is significantly slower

截屏2024-12-30 16 00 06 截屏2024-12-30 16 00 27 截屏2024-12-30 16 01 02

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.

1 participant