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

feat: streamer and member nodes #524

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

feat: streamer and member nodes #524

wants to merge 14 commits into from

Conversation

Mikelle
Copy link
Member

@Mikelle Mikelle commented Dec 9, 2024

Describe your changes

Added:

  • The Streamer is responsible for streaming payloads to member nodes, allowing them to apply these payloads to their respective Geth instances.
  • The Member is connecting to the Streamer to receive payloads from the stream and apply them to their Geth instances

Refactored:

  • Block builder
  • State manager

Issue ticket number and link

Fixes #499

Checklist before requesting a review

  • I have added tests that prove my fix is effective or that my feature works
  • I have made corresponding changes to the documentation

@Mikelle Mikelle self-assigned this Dec 9, 2024
@Mikelle Mikelle changed the title Feat/499/member nodes feat: relay and member nodes Dec 9, 2024
@shaspitz
Copy link
Contributor

shaspitz commented Dec 9, 2024

We should consider alternative naming to "relay" and/or "relayer". Relay already has meaning within the mev-boost pipeline, and relayer already has meaning in our standard bridging protocol (along with other bridging protocols such as IBC)

@Mikelle
Copy link
Member Author

Mikelle commented Dec 9, 2024

@shaspitz Will the name PayloadStreamer work?

@Mikelle Mikelle force-pushed the feat/499/member-nodes branch from bc18cf2 to c900aaf Compare December 9, 2024 17:36
@shaspitz
Copy link
Contributor

shaspitz commented Dec 9, 2024

@shaspitz Will the name PayloadStreamer work?

SGTM!

@Mikelle Mikelle force-pushed the feat/499/member-nodes branch from 4b8999d to 70ccfff Compare December 9, 2024 21:24
@Mikelle Mikelle force-pushed the feat/499/member-nodes branch from 5a2d7c7 to 74f9b77 Compare December 10, 2024 13:39
@Mikelle Mikelle changed the title feat: relay and member nodes feat: streamer and member nodes Dec 10, 2024
@Mikelle Mikelle requested a review from shaspitz December 10, 2024 14:11
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.

Streamer for the member nodes
2 participants