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

Execution Layer P2P Breakout #3 #858

Closed
smartprogrammer93 opened this issue Aug 31, 2023 · 2 comments
Closed

Execution Layer P2P Breakout #3 #858

smartprogrammer93 opened this issue Aug 31, 2023 · 2 comments

Comments

@smartprogrammer93
Copy link

Meeting Info

📆 Subscribe to the Ethereum Protocol Call calendar for calendar invites

Agenda

@smartprogrammer93
Copy link
Author

Meeting summary:

  • EIP draft was presented: eth/69: available-blocks-extended protocol handshake
  • @lightclient expressed concern around this EIP signaling client teams to prune older legacy blocks (which Nethermind is already doing through AncientBodiesBarrier and AncientReceiptBarrier. @jflo mentioned that Besu is also doing that with CheckpointSync). @lightclient was concerned about how this would affect the chain where only a handful of nodes have all the blocks and finding the node with all the blocks would be harder through discovery.
  • I discussed another concern from @holgerd77 that nodes could start disconnecting peers that do not have the block range they need. This would result in the nodes possibly trying to sync being disconnected since they dont have the needed range by others. This can be addressed in the EIP by explicitly mentioning that nodes must not disconnect peers that do not have the range they need.
  • @lightclient and @jflo will ask people from their team to review the draft and comment with any concerns they have.

@holgerd77
Copy link

Thanks for discussing the concerns I expressed 🙏, couldn't unfortunately join the call directly.

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

No branches or pull requests

3 participants