[
https://issues.apache.org/jira/browse/HBASE-25911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17355742#comment-17355742
]
Hudson commented on HBASE-25911:
--------------------------------
Results for branch master
[build #312 on
builds.a.o|https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/master/312/]:
(/) *{color:green}+1 overall{color}*
----
details (if available):
(/) {color:green}+1 general checks{color}
-- For more information [see general
report|https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/master/312/General_20Nightly_20Build_20Report/]
(/) {color:green}+1 jdk8 hadoop3 checks{color}
-- For more information [see jdk8 (hadoop3)
report|https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/master/312/JDK8_20Nightly_20Build_20Report_20_28Hadoop3_29/]
(/) {color:green}+1 jdk11 hadoop3 checks{color}
-- For more information [see jdk11
report|https://ci-hadoop.apache.org/job/HBase/job/HBase%20Nightly/job/master/312/JDK11_20Nightly_20Build_20Report_20_28Hadoop3_29/]
(/) {color:green}+1 source release artifact{color}
-- See build output for details.
(/) {color:green}+1 client integration test{color}
> Fix uses of System.currentTimeMillis (should be
> EnvironmentEdgeManager.currentTime)
> -----------------------------------------------------------------------------------
>
> Key: HBASE-25911
> URL: https://issues.apache.org/jira/browse/HBASE-25911
> Project: HBase
> Issue Type: Sub-task
> Reporter: Andrew Kyle Purtell
> Assignee: Andrew Kyle Purtell
> Priority: Minor
> Fix For: 3.0.0-alpha-1, 2.5.0
>
>
> We introduced EnvironmentEdgeManager a long time ago as a way to inject
> alternate clocks (gettimeofday() aka System.currentTimeMillis()) for unit
> tests. In order for this to be effective, all callers that would otherwise
> use System.currentTimeMillis() must call EnvironmentEdgeManager.currentTime()
> instead, except obviously the implementors of EnvironmentEdge.
> It's common for contributors to be unaware of this practice and reviewers
> might not catch it.
> It will be much more important to have EnvironmentEdgeManager in use where
> expected once we have EnvironmentEdge also providing a monotonic clock
> source. (See parent.)
> On another subtask I will introduce a build enforcer that bans
> System.currentTimeMillis() except where annotated to allow it.
>
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
