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

ADD: implements StoredVerkleTrie #6

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

thomas-quadratic
Copy link
Contributor

PR description

This PR adds a new StoredVerkleTrie class representing a VerkleTrie where Nodes can be fetched from storage as needed while walking through the Trie.

Currently, there is a SimpleVerkleTrie which can be constructed in memory only, and there is a StoredNodeFactory which can retrieve a single node from storage. However, there was no mechanism to walk a VerkleTrie where nodes are stored in storage. StoredVerkleTrie fills this gap.

StoredVerkleTrie is a VerkleTrie where Nodes can be fetched from a database.
It takes a nodeLoader to interface with the database (or any other storage).

Signed-off-by: Thomas Zamojski <[email protected]>
Copy link

github-actions bot commented Nov 7, 2023

  • I thought about the changelog.

@thomas-quadratic thomas-quadratic force-pushed the feature/StoredVerkleTrie branch from 0ccb320 to 26036a7 Compare November 7, 2023 10:24
@matkt matkt merged commit d4ea40c into hyperledger:main Nov 7, 2023
4 checks passed
dragan2234 pushed a commit to dragan2234/besu-verkle-trie that referenced this pull request Jan 22, 2024
StoredVerkleTrie is a VerkleTrie where Nodes can be fetched from a database.
It takes a nodeLoader to interface with the database (or any other storage).

Signed-off-by: Thomas Zamojski <[email protected]>
Signed-off-by: Dragan Pilipovic <[email protected]>
dragan2234 pushed a commit to dragan2234/besu-verkle-trie that referenced this pull request Jan 22, 2024
StoredVerkleTrie is a VerkleTrie where Nodes can be fetched from a database.
It takes a nodeLoader to interface with the database (or any other storage).

Signed-off-by: Thomas Zamojski <[email protected]>
Signed-off-by: Dragan Pilipovic <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants