Skip to content

Conversation

@kevaundray
Copy link

Issue Addressed

Which issue # does this PR address?

Proposed Changes

Please list or describe the changes introduced by this PR.

Additional Info

Please provide any additional information. For example, future considerations
or information useful for reviewers.

@kevaundray
Copy link
Author

We likely also want to remove "already_seen" from ProofsByRoot since that is only going to be used for historical blocks, ie nodes can just send us everything. I was hoping to be able to ask one node for one proof and then another node for a different proof for the same block, but I think we can just apply this optimization afterthefact, moreso given the networking folks will likely have better takes on what to do here

@kevaundray kevaundray force-pushed the kw/exec-proofs-by-range-v2 branch from fe4097c to f67b667 Compare December 22, 2025 16:49
@kevaundray kevaundray force-pushed the kw/exec-proofs-by-range-v2 branch from 70c5ef2 to 24a78ad Compare December 22, 2025 21:44
@kevaundray
Copy link
Author

Although we have ExecutionProofsByRange the missing component is integrating it into range sync which seems a bit involved

@kevaundray
Copy link
Author

Something to flesh out: How do zk attesters interact with non-zk attesters on the network? Does the network topology overlap?

Notice that if all zk attesters only talk to zk attesters and they all don't have the proof, then range sync does not kick in, because no node will be far ahead

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