Skip to content

Conversation

@Mikelle
Copy link
Contributor

@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
Contributor 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

3 participants