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

chore: add root_with_updates method to sparse trie #14393

Merged
merged 1 commit into from
Feb 11, 2025
Merged

chore: add root_with_updates method to sparse trie #14393

merged 1 commit into from
Feb 11, 2025

Conversation

Rjected
Copy link
Member

@Rjected Rjected commented Feb 11, 2025

This allows us to have only one expect in run_sparse_trie, and introduces storage_trie_updates which is used to get storage trie updates

@Rjected Rjected added C-enhancement New feature or request A-trie Related to Merkle Patricia Trie implementation labels Feb 11, 2025
@Rjected Rjected mentioned this pull request Feb 11, 2025
Merged
@Rjected Rjected added this pull request to the merge queue Feb 11, 2025
Merged via the queue into main with commit 8e479f3 Feb 11, 2025
45 checks passed
@Rjected Rjected deleted the dan/root-with-updates-sparse-trie branch February 11, 2025 15:02
18aaddy pushed a commit to 18aaddy/reth that referenced this pull request Feb 12, 2025
@Rjected @18aaddy
chore: add root_with_updates method to sparse trie (paradigmxyz#14393)
f15091c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shekhirin shekhirin shekhirin approved these changes

@rkrasiuk rkrasiuk Awaiting requested review from rkrasiuk rkrasiuk is a code owner

@mattsse mattsse Awaiting requested review from mattsse mattsse is a code owner

@fgimenez fgimenez Awaiting requested review from fgimenez fgimenez is a code owner

Assignees
No one assigned
Labels
A-trie Related to Merkle Patricia Trie implementation C-enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Rjected @shekhirin