Apache9 commented on pull request #3302:
URL: https://github.com/apache/hbase/pull/3302#issuecomment-850043118


   > > What about System.nanoTime?
   > 
   > @Apache9 System.nanotime is not used to get the system clock. I don't mind 
bringing it into EnvironmentEdge but I don't care about it here. It is out of 
scope for all of this work.
   > 
   > Feel free to file a follow up JIRA, to add it to EnvironmentEdge in the 
future, if you would like.
   
   But sometimes we use System.currentTimeMillis just for calculating elapsed 
time, not for getting system clock, I'm not sure if this is also our goal here, 
to also replace these usages.
   
   In general we should all use System.nanoTime for calculating elapsed time, 
but System.nanoTime is usually slower than System.currentTimeMillis...


-- 
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:
us...@infra.apache.org


Reply via email to