lhotari commented on PR #23930: URL: https://github.com/apache/pulsar/pull/23930#issuecomment-2642159133
> Perhaps this is not the root reason, many rate limiters use `System.nanoTime`, such as guava. We should avoid introducing a complex patch, even though the logic is clear. > > The minimal fix seems correct. @nodece Just a reminder that this patch isn't complex based on code metrics. The problem domain is unusual and the feeling about complexity is mainly about unfamiliarity. This PR is still WIP and I'll try to get this into a reasonable state for final review. -- 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]
