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

feat(l1): support snap capability message GetTrieNodes #1031

Merged
merged 183 commits into from
Nov 8, 2024
Merged

feat(l1): support snap capability message GetTrieNodes #1031

merged 183 commits into from
Nov 8, 2024

Conversation

fmoletta
Copy link
Contributor

@fmoletta fmoletta commented Oct 31, 2024 •
edited
Loading

Motivation
Pass hive snap GetTrieNodes test

Description

  • Add trie method get_node
  • Add snap capability messages GetTrieNodes and TrieNodes
  • Support processing GetTrieNodes message

Is part of #934

fmoletta and others added 30 commits October 22, 2024 13:11
@fmoletta
Corrupted History Fix
02fe744 
This PR re-writes this branch's commits on top of the main branch:
- Implement TrieIterator
- Consume Trie when building TrieIterator
- Implement `Store::iter_accounts` using TrieIterator
@fmoletta
Fix slim encoding for AccountState
a2a2aac
@fmoletta
Fix slim encoding for AccountState
aef7441
@fmoletta
Commit progress
13d9bc3
@fmoletta
Commit progress
0ca3589
@fmoletta
Add snap as supported cap
66b6f90
@fmoletta
Push missing changes
41dcaff
@ElFantasma
Initial peer listen loop
a526c10
@fmoletta
Deprecate RLPEncodeSlim trait and use AccountStateSlim instead
562bc49
@fmoletta
Fix logic
3893b05
@fmoletta
Remove debug print
dbe110f
@fmoletta
Limit response bytes
8b8eacb
@fmoletta
Add test
4ee1790
@fmoletta
Add more tests
d87b1ca
@fmoletta
Cleanup test code + Fix logic + add test
89621bf
@fmoletta
Add test
3a14cdd
@fmoletta
Add test
c354321
@fmoletta
Add test
7abd4f7
@fmoletta
Add test
7cee2b7
@fmoletta
Also fetch limit proof
579309f
@fmoletta
Trim test state 408 -> 100
9fccb07
@fmoletta
Simplify logic
38a7d5b
@ElFantasma
Merge branch 'main' into 840-rlpx-listen-loop
574299b
@fmoletta
Encode accounts while building range so we do not encode twice
704da4c
@fmoletta
Merge branch 'main' of github.com:lambdaclass/lambda_ethereum_rust in…
b55294b 
…to trie_iter
@fmoletta
Clippy
6d5cc45
@fmoletta
Add allow tag
9514037
@fmoletta
Remove comment
a5b3934
@fmoletta
Fix typo
68dcef6
@ElFantasma
Sending eth Status message first
f23077e
@fmoletta fmoletta changed the base branch from main to refactor-node-structure November 8, 2024 18:28
Arkenan
Arkenan approved these changes Nov 8, 2024
View reviewed changes
Copy link
Collaborator

@Arkenan Arkenan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

crates/storage/trie/trie.rs Show resolved Hide resolved
Base automatically changed from refactor-node-structure to main November 8, 2024 19:12
@fmoletta fmoletta added this pull request to the merge queue Nov 8, 2024
Merged via the queue into main with commit 3960a49 Nov 8, 2024
4 checks passed
@fmoletta fmoletta deleted the snap-trie-nodes branch November 8, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Arkenan Arkenan Arkenan approved these changes

@ElFantasma ElFantasma ElFantasma approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fmoletta @Arkenan @ElFantasma