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

Increase buffer capacity for mutableRouterEvents flow within RibEvents #635

Merged

Conversation

RahulDMello
Copy link
Contributor

Increase buffer capacity for mutableRouterEvents flow within RibEvents.

Description:
The RibEvents flow is consumed by the ribtree plugin. However, that consumption can sometimes be slower than production so we were losing a few events. Making a large buffer size helps fix that.

…sIncrease buffer capacity for mutableRouterEvents flow within RibEvents
@CLAassistant
Copy link

CLAassistant commented Jun 22, 2024

CLA assistant check
All committers have signed the CLA.

@@ -13,7 +13,7 @@
# org.gradle.parallel=true

GROUP=com.uber.rib
VERSION_NAME=0.16.3-SNAPSHOT
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't need to bump it now -- the release script bumps it automatically

Choose a reason for hiding this comment

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

Oh, I'll remove it lol

@psteiger
Copy link
Contributor

psteiger commented Jul 2, 2024

@tyvsmith can you help w/ getting this merged, please? I see the following message:

Merging is blocked
The base branch restricts merging to authorized users

I'm using my personal GitHub account.

cc @dmellorahul

@tyvsmith tyvsmith merged commit 5aafed4 into uber:main Jul 3, 2024
4 checks passed
@tyvsmith
Copy link
Member

tyvsmith commented Jul 3, 2024

@tyvsmith can you help w/ getting this merged, please? I see the following message:

Merging is blocked
The base branch restricts merging to authorized users

I'm using my personal GitHub account.

cc @dmellorahul

Done, and fixed the issue with your access.

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.

5 participants