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.

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

Commtiment: Use StateReader instead of PatriciaContext #13504

Open
awskii opened this issue Jan 20, 2025 · 0 comments
Open

Commtiment: Use StateReader instead of PatriciaContext #13504

awskii opened this issue Jan 20, 2025 · 0 comments
Assignees
Labels
Milestone

Comments

@awskii
Copy link
Member

awskii commented Jan 20, 2025

We do use some intermediate structure for reading state by trie: PatriciaContext.
Basically, if we make StateReader interface to be used instead, patricia trie could be used together with any implementation of reader which we have plenty.

Another point is that account serializing #13390 relying on account.Account type which may be used inside account cells instead of bunch of fields (eg balance, nonce, etc) for sake of consistency.
Also gives us ability to keep account node storage root which we discard after hashing of account node with storage subtrie. This root is useful during Proof and Witness building.

@awskii awskii added the erigon3 label Jan 20, 2025
@awskii awskii added this to the 3.0.0-beta2 milestone Jan 20, 2025
@awskii awskii self-assigned this Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants