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

Lars Hofhansl commented on PHOENIX-5677:
----------------------------------------

That's why we have the test-suite :)

Looks like this one is also failing: 
org.apache.phoenix.end2end.index.PartialIndexRebuilderIT.testIndexActiveIfRegionMovesWhileRebuilding

The other 2 or 3 failures might be flappers.

> 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: 50m
>  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