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

Josh Elser commented on ACCUMULO-4425:
--------------------------------------

bq. The label was the easiest first pass I thought of. I could simplify it with 
a boolean conditional on the outer loop. I'm more concerned about the strategy 
of handling this in the test itself than the specific implementation.

Yup, I understand and agree with you completely.

bq. I agree with the concern about runtime issues. That's why I put it up for 
review as a PR. I'm concerned we're not properly handling this internally in 
the WalStateManager. But, I'm also wondering if this is something that can only 
happen in the test. The thing is... in the dirty shutdown case, I'm not 
actually sure why these states persist. Perhaps it's just because the ephemeral 
ZK nodes haven't timed out yet? Maybe it's not something to be concerned about 
in a real system and is only an artifact of the test. At the very least, it's 
clear from the workaround that they will eventually resolve themselves, and 
maybe that's sufficient for a running system? This part of our code is hard to 
reason about... because there aren't a lot of comments explaining how the 
design is supposed to work.

I'm sure [~kturner]

> VolumeIT.testDirtyReplaceVolumes fails
> --------------------------------------
>
>                 Key: ACCUMULO-4425
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4425
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Christopher Tubbs
>            Assignee: Christopher Tubbs
>            Priority: Blocker
>             Fix For: 1.8.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> *Error Message*
> {code}
> Unexpected volume 
> file:/var/lib/jenkins/workspace/Accumulo-1.8-ITs-failures/test/target/mini-tests/org.apache.accumulo.test.VolumeIT_testDirtyReplaceVolumes/volumes/v1/wal/jenkins.revelc.net+38766/3eb39803-c014-4195-943a-7a12efa2f515
> {code}
> *Stacktrace*
> {code}
> java.lang.AssertionError: Unexpected volume 
> file:/var/lib/jenkins/workspace/Accumulo-1.8-ITs-failures/test/target/mini-tests/org.apache.accumulo.test.VolumeIT_testDirtyReplaceVolumes/volumes/v1/wal/jenkins.revelc.net+38766/3eb39803-c014-4195-943a-7a12efa2f515
>       at 
> org.apache.accumulo.test.VolumeIT.verifyVolumesUsed(VolumeIT.java:441)
>       at 
> org.apache.accumulo.test.VolumeIT.testReplaceVolume(VolumeIT.java:533)
>       at 
> org.apache.accumulo.test.VolumeIT.testDirtyReplaceVolumes(VolumeIT.java:566)
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to