This is an automated email from the ASF dual-hosted git repository.
zike pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push:
new 81971e2d91a [fix] [doc] Fix outdated java-doc of rate limiter (#19014)
81971e2d91a is described below
commit 81971e2d91a18ac7aac245c36ce082cd3b8176ad
Author: fengyubiao <[email protected]>
AuthorDate: Tue Apr 11 14:50:08 2023 +0800
[fix] [doc] Fix outdated java-doc of rate limiter (#19014)
### Motivation
After PR #8611, the acquiring permits can be larger than configured
msg-rate if used by subscribing. But doc was not updated in time.
### Modifications
fix the outdated doc
---
.../apache/pulsar/broker/service/persistent/DispatchRateLimiter.java | 2 --
1 file changed, 2 deletions(-)
diff --git
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/DispatchRateLimiter.java
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/DispatchRateLimiter.java
index 5b8987580c4..b1e48035484 100644
---
a/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/DispatchRateLimiter.java
+++
b/pulsar-broker/src/main/java/org/apache/pulsar/broker/service/persistent/DispatchRateLimiter.java
@@ -99,10 +99,8 @@ public class DispatchRateLimiter {
*/
public boolean tryDispatchPermit(long msgPermits, long bytePermits) {
boolean acquiredMsgPermit = msgPermits <= 0 ||
dispatchRateLimiterOnMessage == null
- // acquiring permits must be < configured msg-rate;
|| dispatchRateLimiterOnMessage.tryAcquire(msgPermits);
boolean acquiredBytePermit = bytePermits <= 0 ||
dispatchRateLimiterOnByte == null
- // acquiring permits must be < configured msg-rate;
|| dispatchRateLimiterOnByte.tryAcquire(bytePermits);
return acquiredMsgPermit && acquiredBytePermit;
}