[GitHub] [hbase] apurtell edited a comment on pull request #3302: HBASE-25911 Replace calls to System.currentTimeMillis with EnvironmentEdgeManager.currentTime

2021-05-28 Thread GitBox
apurtell edited a comment on pull request #3302: URL: https://github.com/apache/hbase/pull/3302#issuecomment-850569022 > 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

[GitHub] [hbase] apurtell edited a comment on pull request #3302: HBASE-25911 Replace calls to System.currentTimeMillis with EnvironmentEdgeManager.currentTime

2021-05-27 Thread GitBox
apurtell edited a comment on pull request #3302: URL: https://github.com/apache/hbase/pull/3302#issuecomment-849872794 > 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