-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Closed
Labels
A-trieRelated to Merkle Patricia Trie implementationRelated to Merkle Patricia Trie implementationC-perfA change motivated by improving speed, memory usage or disk footprintA change motivated by improving speed, memory usage or disk footprint
Description
Describe the feature
Currently, every time we have a new block, we create an empty SparseTrie for the accounts trie to reveal new proofs and apply updates in there.
After #15637, still at least half of the time in reveal_account_proof is spent reserving the space for new nodes to insert
This can be optimized by not de-allocating the accounts trie
reth/crates/trie/sparse/src/state.rs
Lines 28 to 29 in ab85300
| /// Sparse account trie. | |
| state: SparseTrie<F::AccountNodeProvider>, |
and instead just clear it and carry over to the next state root task invocation.
Additional context
No response
Metadata
Metadata
Assignees
Labels
A-trieRelated to Merkle Patricia Trie implementationRelated to Merkle Patricia Trie implementationC-perfA change motivated by improving speed, memory usage or disk footprintA change motivated by improving speed, memory usage or disk footprint
Type
Projects
Status
Completed