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

Add factory method to RateLimiter to allow setting maxBurstSeconds #1581

Open
gissuebot opened this issue Oct 31, 2014 · 1 comment
Open

Add factory method to RateLimiter to allow setting maxBurstSeconds #1581

gissuebot opened this issue Oct 31, 2014 · 1 comment
Labels
P4 no SLO package=concurrent status=research type=enhancement Make an existing feature better

Comments

@gissuebot
Copy link

gissuebot commented Oct 31, 2014 •
edited by cgdecker
Loading

Original issue created by [email protected] on 2013-11-16 at 07:44 PM

When someone is using the RateLimiter to throttle requests, it may be desirable never to exceed the permitsPerSecond, even if that results in underutilization of resources. To achieve that the maxBurstSeconds must be set to 0.

Currently even though the desired functionality is supported by the class internally, it is not exposed by the RateLimiter's public API
@gissuebot
Copy link
Author

Original comment posted by [email protected] on 2013-11-20 at 08:13 PM

(No comment entered for this change.)

Status: Research
Labels: Type-Enhancement, Package-Concurrent

@cgdecker cgdecker removed the migrated label Nov 1, 2014
@farmerjohngit farmerjohngit mentioned this issue Jan 22, 2019
Open
@raghsriniv raghsriniv added the P4 no SLO label Aug 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
P4 no SLO package=concurrent status=research type=enhancement Make an existing feature better
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cgdecker @gissuebot @raghsriniv