StevenLuMT opened a new pull request, #3501: URL: https://github.com/apache/bookkeeper/pull/3501
Descriptions of the changes in this PR: ### Motivation The latency of the OpStatsLogger.registerSuccessfulEvent calculation is to convert the time to milliseconds <img width="855" alt="image" src="https://user-images.githubusercontent.com/42990025/191929953-bdac2d25-3fbe-4697-b3e9-f4942ed7c074.png";> but Counter.add nothing to do so when using Counter for latency statistics, the time unit and OpStatsLogger are not unified, then we unified latency metric unit ### Changes 1. change name : Counter.add --> Counter.addCount 2. add new method Counter.addLatency to count the time and convert the time to milliseconds then how to use counter correctly: 1. when using Counter for latency metric, call Counter.addLatency 2. when using Counter for count metric, call Counter.addCount -- 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]
![https://user-images.githubusercontent.com/42990025/191929953-bdac2d25-3fbe-4697-b3e9-f4942ed7c074.png"](https://user-images.githubusercontent.com/42990025/191929953-bdac2d25-3fbe-4697-b3e9-f4942ed7c074.png"){kind=link}