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

Benchmark SparseTrie::update_rlp_node_level #12012

Open
Tracked by #11167
shekhirin opened this issue Oct 23, 2024 · 0 comments · May be fixed by #12046
Open
Tracked by #11167

Benchmark SparseTrie::update_rlp_node_level #12012

shekhirin opened this issue Oct 23, 2024 · 0 comments · May be fixed by #12046
Assignees
@shekhirin
Labels
A-trie Related to Merkle Patricia Trie implementation C-benchmark A change that impacts how or what we benchmark

Comments

@shekhirin
Copy link
Collaborator

Describe the feature

To be able to measure the effect of #11806, we need to add benchmarks for the RevealedSparseTrie::update_rlp_node_level method. They should have different sizes of tries and different levels of updates.

Additional context

No response
@shekhirin shekhirin added C-test A change that impacts how or what we test C-benchmark A change that impacts how or what we benchmark A-trie Related to Merkle Patricia Trie implementation labels Oct 23, 2024
@shekhirin shekhirin mentioned this issue Oct 23, 2024
Open
@shekhirin shekhirin self-assigned this Oct 23, 2024
@shekhirin shekhirin changed the title Benchmark RevealedSparseTrie::update_rlp_node_level Benchmark SparseTrie::update_rlp_node_level Oct 23, 2024
@shekhirin shekhirin linked a pull request Oct 24, 2024 that will close this issue
Open
@shekhirin shekhirin removed the C-test A change that impacts how or what we test label Oct 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shekhirin shekhirin

Labels
A-trie Related to Merkle Patricia Trie implementation C-benchmark A change that impacts how or what we benchmark
Projects
Reth Tracker
Status: Todo
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bench(trie): RevealedSparseTrie::update_rlp_node_level paradigmxyz/reth
1 participant
@shekhirin