aloyszhang edited a comment on pull request #10384:
URL: https://github.com/apache/pulsar/pull/10384#issuecomment-842292133


   > It feels that the 1 second rate limiting interval would have to be shared 
between the 2 different rate limiters to fix the inconsistencies when both 
limits are set.
   
   @lhotari  Yes,  inconsistent between `publishThrottlingRateInMsg` and 
`publishThrottlingRateInByte` may happend.
   Totally agree with that  `publishThrottlingRateInMsg` and 
`publishThrottlingRateInByte` should share  one  same `rateTime ` and refresh 
ticket at the same time.


-- 
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to