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

EIP-7594: clarifying reconstruction and cross-seeding #3794

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

cskiraly
Copy link
Contributor

Attempt to clarify rules and recommendations for on-the-fly reconstruction and cross-seeding:

  • make reconstruction a SHOULD if a node has enough data,
  • allos delaying reconstruction,
  • specify how to corss-seed reconstructed columns for subscribed and for unsubscribed topics.

Notes: For unsubscribed topics, we are not seeding to avoid potentially gererating too many duplicates. If, however, we want to speed up spread in this case, it might make sense to send it to a limited number (e.g. 1) of fanout peers, or to immediately generate IHAVE messages without waiting for the heartbeat.

- when should a node reconstruct
- how to cross-seed for subscribed and unsubscribed topics

Signed-off-by: Csaba Kiraly <[email protected]>
@cskiraly cskiraly changed the title clarifying reconstruction and cross-seeding EIP-7594: clarifying reconstruction and cross-seeding Jun 11, 2024
@cskiraly
Copy link
Contributor Author

@hwwhww , I have created this PR to improve the description of on-the-fly repair and related cross-seeding based on what we've discussed during the interop. I hope it maps well to the ideas that emerged. can you label this with eip-7594? Thanks!

@jtraglia jtraglia added the EIP-7594 PeerDAS label Jun 11, 2024
Copy link
Collaborator

@dapplion dapplion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New text sounds good me

Copy link
Member

@ppopth ppopth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@hwwhww hwwhww merged commit 258c2c9 into ethereum:dev Jun 27, 2024
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EIP-7594 PeerDAS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants