uber / tchannel

network multiplexing and framing protocol for RPC
MIT License
1.15k stars 129 forks source link

Verify that busy frames can be smoothed over #1307

Closed Raynos closed 9 years ago

Raynos commented 9 years ago

This uses the TimeSeriesCluster to create a test case that generates 25 req/s per HyperbahnWorker.

We set the rateLimit to 10 req/s for one out of 10 Hyperbahn workers and then assert that zero requests fail due to retries.

We also assert that the rate limiter actually kicked in.

r: @kriskowal @rf

cc @anson627 This proofs that retries on Busy solve our problem. We still need to debug why xlate didn't do retries.

anson627 commented 9 years ago

awesome! Then I can port TimeSeriesCluster to xlate and do the same test.

Raynos commented 9 years ago

@anson627 put me on the review for that. That's something I can help with.

kriskowal commented 9 years ago

Please leave some notes for future-@raynos et al, in comments or markdown, for the usage of this demo and, in particular, what makes the first bucket special.

Raynos commented 9 years ago

@kriskowal will do.

Raynos commented 9 years ago

This PR will need to be rebased in github.com/uber/hyperbahn