[ 
https://issues.apache.org/jira/browse/PHOENIX-5677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17022473#comment-17022473
 ] 

Hudson commented on PHOENIX-5677:
---------------------------------

SUCCESS: Integrated in Jenkins build Phoenix-4.x-HBase-1.5 #255 (See 
[https://builds.apache.org/job/Phoenix-4.x-HBase-1.5/255/])
PHOENIX-5677 - Replace System.currentTimeMillis with (gjacoby: rev 
b5a5ceb9c01b8b54f271fc06dae2ba88e432fe1e)
* (edit) 
phoenix-core/src/main/java/org/apache/phoenix/mapreduce/index/IndexScrutinyMapperForTest.java
* (edit) 
phoenix-core/src/it/java/org/apache/phoenix/end2end/index/PartialIndexRebuilderIT.java


> Replace System.currentTimeMillis with EnvironmentEdgeManager in non-test code
> -----------------------------------------------------------------------------
>
>                 Key: PHOENIX-5677
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-5677
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Geoffrey Jacoby
>            Assignee: Geoffrey Jacoby
>            Priority: Major
>             Fix For: 5.1.0, 4.16.0
>
>         Attachments: PHOENIX-5677-4.x-HBase-1.3.patch, 
> PHOENIX-5677-addendum-4.x-HBase-1.3.patch
>
>          Time Spent: 1h
>  Remaining Estimate: 0h
>
> Phoenix is inconsistent in using either system clock or 
> EnvironmentEdgeManager to get current time. The EnvironmentEdgeManager is 
> occasionally very useful in tests to control time deterministically without 
> needing to sleep. Direct references to System.currentTimeMillis in non-test 
> code should be switched over. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to