areyouok commented on pull request #3659:
URL: https://github.com/apache/rocketmq/pull/3659#issuecomment-998511009


   > I found `System.nanoTime()` was used instead of 
`System.currentTimeMillis()`, will it bring benefits to performance 
improvement? It also brings some conversion between nanosecond variables and 
config values defined in millisecond.
   
   There is no performance benefit. 
   
   nanoTime() are more strictly than currentTimeMillis() for measuring time 
elapse. For example, NTP service may adjust system clock, the nanoTime() method 
will not affect by it. See javadoc of the method.


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


Reply via email to