Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-17239

Race in CompactionExecutorTest

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Normal
    • Resolution: Fixed
    • 3.0.26, 3.11.12, 4.0.2
    • Test/unit
    • None

    Description

      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.

      Attachments

        Activity

          People

            jonmeredith Jon Meredith
            jonmeredith Jon Meredith
            Jon Meredith
            Josh McKenzie
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 50m
                50m