feat(l2): multiple provers in file #1477
Open
Conversation
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
…to l2/multiple_provers_in_file
fborello-lambda
force-pushed
the
l2/multiple_provers_in_file
branch
from
a4f2032 to
992191a
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Motivation
To support multiple proving systems, we may want to store the proofs and wait to have all the needed information to send the transaction to the L1 and verify them on chain.
In the previous PR (#1466) a HashMap was proposed to store the proofs in memory, a more robust solution is needed in case the L2 node goes down. In order to avoid re-execution, the proofs can be saved in files.
Description
Closes #1060