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

EPIC: p2p stack refactor #1531

Open
evan-forbes opened this issue Nov 18, 2024 · 0 comments
Open

EPIC: p2p stack refactor #1531

evan-forbes opened this issue Nov 18, 2024 · 0 comments
Assignees
Labels
p2p refactor WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧

Comments

@evan-forbes
Copy link
Member

evan-forbes commented Nov 18, 2024

Currently, the p2p cannot allocate bandwidth dynamically or cache incoming messages to process in parallel.

While the crucial details such as transport have not yet been determined, we know we want these properties.

  • Dynamic allocation of bandwidth
    • This means that if there are message to send, and two connected peers have bandwidth available, it is utilized.
  • Cache incoming message without blocking
    • Currently, the p2p stack has the potential to block reading from the tcp buffer as each message is processed serially. Instead, each message should avoid blocking to the extent that it can by the raw bytes immediately being passed to a reactor, or better yet a channel.

Options

There are several different optoins we could take.

Do determine which stack is ideal, we should first complete:

then reevaluate

@evan-forbes evan-forbes added WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧 labels Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p2p refactor WS: Big Blonks 🔭 Improving consensus critical gossiping protocols WS: Maintenance 🔧
Projects
None yet
Development

No branches or pull requests

2 participants