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.

Sign up for GitHub

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

Jump to bottom

use multiple streams to reduce the amount of useless message processi… #56

Merged
merged 2 commits into from
Dec 7, 2023
Merged

use multiple streams to reduce the amount of useless message processi… #56

merged 2 commits into from
Dec 7, 2023

Conversation

k-rister
Copy link
Contributor

@k-rister k-rister commented Dec 6, 2023

…ng for the participants

  • using 3 streams: global, leader, and followers

  • the leader only listens to the global and leader streams

  • the followers only listen to the global and followers streams

  • this allows participants to avoid processing messages that are not intended for them -- ie. the followers don't need to process messages that other followers send to the leader

@k-rister
use multiple streams to reduce the amount of useless message processi…
62effa9 
…ng for the participants

- using 3 streams: global, leader, and followers

- the leader only listens to the global and leader streams

- the followers only listen to the global and followers streams

- this allows participants to avoid processing messages that are not
  intended for them -- ie. the followers don't need to process
  messages that other followers send to the leader
@k-rister k-rister self-assigned this Dec 6, 2023
atheurer
atheurer reviewed Dec 7, 2023
View reviewed changes
Copy link
Contributor

@atheurer atheurer left a comment

Choose a reason for hiding this comment

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

This is completely transparent to the RB calls by followers and leaders in the endpoint and engine code?

@k-rister
Copy link
Contributor Author

k-rister commented Dec 7, 2023

This is completely transparent to the RB calls by followers and leaders in the endpoint and engine code?

Yes

@k-rister k-rister merged commit afd44cc into master Dec 7, 2023
147 checks passed
@k-rister k-rister deleted the scalability branch December 7, 2023 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@atheurer atheurer atheurer approved these changes

@rafaelfolco rafaelfolco Awaiting requested review from rafaelfolco

Assignees

@k-rister k-rister

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@k-rister @atheurer