waives / waives.net

The .NET SDK for Waives
https://docs.waives.io/docs/dotnet-sdk-overview
MIT License
0 stars 0 forks source link

Remove timer from rate limiter #15

Closed phildrip closed 5 years ago

phildrip commented 6 years ago

Is your feature request related to a problem? Please describe. A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Describe the solution you'd like A clear and concise description of what you want to happen.

Describe alternatives you've considered A clear and concise description of any alternative solutions or features you've considered.

Additional context Add any other context or screenshots about the feature request here.

alastairs commented 5 years ago

We might be able to remove the rate limiter entirely and replace it with a bulkhead isolation policy from Polly, as described on the Polly wiki.

Although it's only just kicked off, there's a mutable bulkhead isolation policy in Polly-Contrib. This looks as though it will allow for dynamic capacity adjustment.