search

uber-archive / hyperbahn

Service discovery and routing for large scale microservice operations
MIT License
396 stars 57 forks source link

Correctly handle rpsLimits set to zero... #251

Closed jcorbin closed 8 years ago

jcorbin commented 8 years ago

...by effectively disabling rate limits.

I noticed that our test suite at least was provoking rate limiting for total rps limit set to 0. Whether or not the tests should be running with rps limit 0, I feel this is more correct implementation.

r @raynos @rf

Raynos commented 8 years ago

lgtm.

Raynos commented 8 years ago

lgtm.