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

Jon Meredith commented on CASSANDRA-17239:
------------------------------------------

Test failures unrelated to CompactionExecutorTest - going ahead with merge

> Race in CompactionExecutorTest
> ------------------------------
>
>                 Key: CASSANDRA-17239
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-17239
>             Project: Cassandra
>          Issue Type: Bug
>          Components: Test/unit
>            Reporter: Jon Meredith
>            Assignee: Jon Meredith
>            Priority: Normal
>             Fix For: 3.0.x, 3.11.x, 4.0.x
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> CompactionExecutorTest has a race between the runnable/callable under test 
> completing
> and the {{afterExecute}} method stashing it for the test.  Replace the 
> wait/sleep loop
> with a {{SimpleCondition}} that is signaled once the test task throwable has 
> been recorded.
> This seems fairly hard to hit but has happened on CI.  It took about 2600 
> iterations on my MacBook to trigger, but you can artificially hit frequently 
> by adding a sleep at the start of the afterExecute method.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to