[ https://issues.apache.org/jira/browse/CASSANDRA-17239?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17469978#comment-17469978 ]
Josh McKenzie commented on CASSANDRA-17239: ------------------------------------------- +1. Had a question about whether inheriting the 10 second timeout w/out revising it makes sense from a "brittleness and future containerized test execution environments will make us sad" perspective but still; this should fix it. > 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: 20m > 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