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

db: cache chain config in local kv::api #2798

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

Conversation

battlmonstr
Copy link
Contributor

It is possible to cache the ChainConfig structure after reading it from mdbx, because it is immutable after the node has started.

This optimization shaves off 0,6% of total time during stress_test_eth_call_20M.tar.

Screenshot 2025-03-18 at 13 05 00

@battlmonstr battlmonstr requested review from canepat and lupin012 March 18, 2025 12:26
@battlmonstr battlmonstr force-pushed the pr/chain_config_cache branch from 52872de to bd0e9ba Compare March 19, 2025 13:11
@battlmonstr battlmonstr enabled auto-merge (squash) March 19, 2025 13:11
@battlmonstr battlmonstr force-pushed the pr/chain_config_cache branch 2 times, most recently from de8853e to 6dd33c2 Compare March 21, 2025 10:56
@battlmonstr battlmonstr force-pushed the pr/chain_config_cache branch from 6dd33c2 to 57237ce Compare March 21, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant