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

perf(engine): migrate to AsyncStateRoot #10892

Closed
rkrasiuk opened this issue Sep 13, 2024 · 0 comments · Fixed by #10927
Closed

perf(engine): migrate to AsyncStateRoot #10892

rkrasiuk opened this issue Sep 13, 2024 · 0 comments · Fixed by #10927
Assignees
@fgimenez
Labels
A-consensus Related to the consensus engine A-trie Related to Merkle Patricia Trie implementation C-perf A change motivated by improving speed, memory usage or disk footprint

Comments

@rkrasiuk
Copy link
Member

Description

Recently, we've introduced ParallelStateRoot in the tree (ref #10666). AsyncStateRoot is more performant version of that (refer to the docs). Swap ParallelStateRoot for AsyncStateRoot in the engine tree code.
@rkrasiuk rkrasiuk added C-perf A change motivated by improving speed, memory usage or disk footprint A-consensus Related to the consensus engine A-trie Related to Merkle Patricia Trie implementation labels Sep 13, 2024
@rkrasiuk rkrasiuk changed the title perf(tree): migrate to AsyncStateRoot perf(engine): migrate to AsyncStateRoot Sep 13, 2024
@rkrasiuk rkrasiuk mentioned this issue Sep 15, 2024
Merged
@fgimenez fgimenez mentioned this issue Sep 16, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@fgimenez fgimenez

Labels
A-consensus Related to the consensus engine A-trie Related to Merkle Patricia Trie implementation C-perf A change motivated by improving speed, memory usage or disk footprint
Projects
Reth Tracker
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

perf(engine): migrate to AsyncStateRoot paradigmxyz/reth
2 participants
@fgimenez @rkrasiuk