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

Feature suggestion: Token Bucket and Leaky Bucket #528

Closed
sq735 opened this issue Nov 7, 2018 · 2 comments
Closed

Feature suggestion: Token Bucket and Leaky Bucket #528

sq735 opened this issue Nov 7, 2018 · 2 comments
Labels

Comments

@sq735
Copy link

sq735 commented Nov 7, 2018

Is it possible to add in a new version "Token Bucket" and "Leaky Bucket" policy?

@sq735 sq735 changed the title Token Bucket and Leaky Bucket Feature suggestion: Token Bucket and Leaky Bucket Nov 7, 2018
@reisenberger
Copy link
Member

Hi @sq735 . This is not something I have any current time available to develop myself. (The time I have for Polly will likely focus on Distributed Circuit Breaker and metrics.)

We are fine with taking Pull Requests though, if you (or anyone else) wants to take this on. (Right now we have great contributors developing InjectFault and InjectLatency, demonstrating how easy it is to add new policies.)

See also the existing discussion in the Polly roadmap around rate-limiting policies with some other algorithm options, and related #330 .

@reisenberger
Copy link
Member

To avoid noise on the 'up-for-grabs' labels (three similar issues), I'm closing this issue in favour of the similar #260 (and noting the ideas of Token Bucket and Leaky Bucket there).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants