[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-27 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r677262895 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ## @@ -257,14 +271,15 @@ public synchronized TimeUnit getRa

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-27 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r677262895 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ## @@ -257,14 +271,15 @@ public synchronized TimeUnit getRa

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-26 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676505426 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PublishRateLimiterTest.java ## @@ -104,9 +104,13 @@ public void testPreci

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-26 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676356713 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PublishRateLimiterTest.java ## @@ -104,9 +104,13 @@ public void testPreci

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-26 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676341281 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PublishRateLimiterTest.java ## @@ -104,9 +104,13 @@ public void testPreci

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-25 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676317299 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ## @@ -257,14 +271,15 @@ public synchronized TimeUnit getRa

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-25 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676315712 ## File path: pulsar-broker/src/test/java/org/apache/pulsar/broker/service/PublishRateLimiterTest.java ## @@ -104,9 +104,13 @@ public void testPreci

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-25 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676112146 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ## @@ -257,14 +267,14 @@ public synchronized TimeUnit getRa

[GitHub] [pulsar] danielsinai commented on a change in pull request #11446: [issue #13351] Solving precise rate limiting does not takes effect

2021-07-24 Thread GitBox
danielsinai commented on a change in pull request #11446: URL: https://github.com/apache/pulsar/pull/11446#discussion_r676026166 ## File path: pulsar-common/src/main/java/org/apache/pulsar/common/util/RateLimiter.java ## @@ -257,14 +267,14 @@ public synchronized TimeUnit getRa