-
Notifications
You must be signed in to change notification settings - Fork 283
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
feat: persistent archiver store #3410
Merged
Merged
Changes from 11 commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
5ba9aff
feat: add jest helpers to @aztec/types
alexghr f3869b6
refactor: split MemoryArchiverStore to own file
alexghr de93bfc
refactor: addLogs per block
alexghr 051549d
test: add comprehensive archiver_store test suite
alexghr 0b378e4
feat: add archiver store backed by LMDB
alexghr bdf9c5c
fix: await contracts added to the archiver store
alexghr 1b82206
feat: use persistent archiver store
alexghr d21cb49
Merge branch 'master' into alexg/feat/persistent-archiver-store
alexghr 02f924b
fix: L1ToL2Message.random takes optional entryKey
alexghr d84ab77
fix: typing error in ts-jest
alexghr fd95142
fix: parse aztec address from string
alexghr b33ce02
refactor: add loop labels for more elegant breaks
alexghr ca43db3
refactor: use of transactions
alexghr 8fc557e
Merge branch 'master' into alexg/feat/persistent-archiver-store
alexghr 8653839
fix: merkle trees store last globalVariablesHash
alexghr a4235f9
fix: store/restore indexed tree's leaves to db
alexghr 20c5acd
fix: pass global vars hash
alexghr 5e202ef
refactor: inject archiver store into archiver
alexghr b0955a4
refactor: archiver gets L1 block number from store
alexghr 600002b
refactor: use private property for l1BlockNumber
alexghr f724acd
Merge branch 'master' into alexg/feat/persistent-archiver-store
alexghr 8a8a60b
refactor: segregate tx and contract indexed data
alexghr beb0102
refactor: add debug logging to lmdb archiver store
alexghr ad2093a
fix: use uint32 for block keys
alexghr 4010d8a
fix: process L1->L2 messages just once
alexghr 9b46e88
refactor: store entry keys as buffers in dup table
alexghr 04e7e73
Merge remote-tracking branch 'origin/master' into alexg/feat/persiste…
alexghr 4a85244
fix: memory l1tol2 message store is idempotent
alexghr aa07751
test: verify archiver store is idempotent
alexghr 1cb5e12
docs: update archiver sync doc
alexghr 801fb6f
fix: slow tree test more lenient
alexghr File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've left the old MemoryArchiverStore in as a reference for the code review and in case we might need it for other tests but otherwise all code paths should pass a
rootDB
now and use a temporary LMDB instance if persistance isn't needed.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm wondering if this function should just accept the ArchiverStore interface and the concrete implementation is created at node setup. Otherwise, if I wanted to add a different DB implementation I would need to pollute the arguments.