Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

broker: allow further banking of unused flow-control credit #340

Merged
merged 1 commit into from
Jun 27, 2023

Conversation

jgraettinger
Copy link
Contributor

@jgraettinger jgraettinger commented Jun 27, 2023

Allow up to ten seconds of unused flow-control credit to be banked against a future burst of traffic.

This is in service of workloads where clients want to write short burts of data and then go away for a while.


This change is Reviewable

Allow up to ten seconds of unused flow-control credit to be banked
against a future burst of traffic.

This is in service of workloads where clients want to write short
burts of data and then go away for a while.
@jgraettinger jgraettinger requested a review from psFried June 27, 2023 06:38
Copy link
Contributor

@psFried psFried left a comment

Choose a reason for hiding this comment

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

LGTM

@jgraettinger jgraettinger merged commit fc3ba77 into master Jun 27, 2023
@jgraettinger jgraettinger deleted the johnny/flow-control-burst branch June 27, 2023 14:27
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants