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.

Sign up for GitHub

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

Jump to bottom

Implement Forward Sync for EIP-4844 #6669

Closed
Tracked by #5681
StefanBratanov opened this issue Jan 9, 2023 · 0 comments · Fixed by #6684
Closed
Tracked by #5681

Implement Forward Sync for EIP-4844 #6669

StefanBratanov opened this issue Jan 9, 2023 · 0 comments · Fixed by #6684
Assignees
@StefanBratanov
Labels
Epic Deneb Issues required to implement Deneb upgrade

Comments

@StefanBratanov
Copy link
Contributor

StefanBratanov commented Jan 9, 2023 •
edited
Loading

Implement new sync logic PeerSync and BatchSync:

  • for pre 4844 blocks we continue using BeaconBlocksByRange
  • for post 4844 blocks:
    • if we need blocks and blobs we use BeaconBlocksByRange and BlobsSidecarsByRange (this could be done by let SyncSourceBatch handle blocks and blobs and check the consistency, having two different batch types seems more complicated)
    • if we need only blocks we continue using BeaconBlocksByRange
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@StefanBratanov StefanBratanov

Labels
Epic Deneb Issues required to implement Deneb upgrade
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement forward multipeer sync for EIP-4844 StefanBratanov/teku
1 participant
@StefanBratanov