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

L2msgs in single tx #178

Closed
wants to merge 4,266 commits into from
Closed

L2msgs in single tx #178

wants to merge 4,266 commits into from

Conversation

ImJeremyHe
Copy link
Member

No description provided.

diegoximenes and others added 30 commits June 26, 2024 12:14
…ersion

Require program.version == params.Version when caching
…_split_l1gas

[NIT-2558] Consensus/Execution split: simplifies ConsensusSequencer and ExecutionEngine interfaces
…sig-checking

Remove sig from DAS writer iface, sign in client
pin espresso-sequencer-go to new release v0.0.20
The logic for fetching keysets and batches was originally all done using
GetByHash on the the DASReader interface. Nitro nodes, batch posters,
and daservers would all include the ChainFetchReader at the beginning
of their chain of DASReaders in order to intercept GetByHash calls and
do the lookup of the keyset on L1. It was confusing, however, because it
also forwarded GetByHash requests to an inner DASReader, since it had no
way of telling if the GetByHash requests were for keysets or batch data.

This commit renames the ChainFetchReader to KeysetFetcher and removes the
inner DASReader and chaining logic so that it only tries to fetch keysets
from L1 or its own internal cache. The orchestration of normal nodes and
batch posters is changed to set up the KeysetFetcher separately rather
than part of the chain of DASReaders.

The replay binary preimage reader still needs to be updated.
@ImJeremyHe ImJeremyHe closed this Aug 5, 2024
@CLAassistant
Copy link

CLAassistant commented Aug 5, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 6 committers have signed the CLA.

✅ sveitser
✅ Sneh1999
❌ PlasmaPower
❌ renlulu
❌ joshuacolvin0
❌ Tristan-Wilson
You have signed the CLA already but the status is still pending? Let us recheck it.

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.