platinummonkey / go-concurrency-limits

Go implementation of Netflix/concurrency-limits
Apache License 2.0
121 stars 21 forks source link

Fix gradient limit calculation #166

Closed jhalterman closed 1 month ago

jhalterman commented 1 month ago

I noticed that the gradient limit was very high when latency was increasing and the gradient was .5. This PR fixes what appears to have been a typo in the limit calculation, compared to the concurrency-limits implementation.

platinummonkey commented 1 month ago

Thanks for the fix! Mind updating the test? 🙏

jhalterman commented 1 month ago

Yes, updated!