maedhroz commented on a change in pull request #1045:
URL: https://github.com/apache/cassandra/pull/1045#discussion_r684584087
##########
File path: test/burn/org/apache/cassandra/transport/SimpleClientPerfTest.java
##########
@@ -190,7 +199,9 @@ public int encodedSize(QueryMessage queryMessage,
ProtocolVersion version)
AtomicBoolean measure = new AtomicBoolean(false);
DescriptiveStatistics stats = new DescriptiveStatistics();
Lock lock = new ReentrantLock();
-
+ RateLimiter limiter = RateLimiter.create(2000);
Review comment:
The intent here was to use the Guava limiter to fix the client at
particular "stable rates" under, at, and above the server-side limit, which is
obviously enforced by `NoWaitRateLimiter`.
Aside: I'm trying to come up with some better names for `NoWaitRateLimiter`.
It's a "rate limiter that defers waiting decisions to the caller and delegates
internals to a throttling model", so something like `DelegatingRateLimiter`
might make sense.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
