Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-28413

Fix race condition in TestCleanerChore.retriesIOExceptionInStatus

Add voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Test
    • Status: Patch Available
    • Major
    • Resolution: Unresolved
    • 2.6.0, 3.0.0-alpha-4, 4.0.0-alpha-1
    • None
    • test

    Description

      We occasionally get a test failure in TestCleanerChore.retriesIOExceptionInStatus. For example, from a recent PR build on branch-2.6,

      java.util.concurrent.ExecutionException: java.io.IOException: whomp whomp.
      	at java.base/java.util.concurrent.CompletableFuture.reportGet(CompletableFuture.java:395)
      	at java.base/java.util.concurrent.CompletableFuture.get(CompletableFuture.java:1999)
      	at org.apache.hadoop.hbase.master.cleaner.TestCleanerChore.retriesIOExceptionInStatus(TestCleanerChore.java:163)
      ...
      Caused by: java.io.IOException: whomp whomp.
      	at org.apache.hadoop.hbase.master.cleaner.TestCleanerChore$1.listStatus(TestCleanerChore.java:134)
      	at org.apache.hadoop.hbase.master.cleaner.CleanerChore.traverseAndDelete(CleanerChore.java:475)
      	at org.apache.hadoop.hbase.master.cleaner.CleanerChore.lambda$chore$0(CleanerChore.java:258)
      	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
      	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
      	... 1 more
      

      This looks like a race condition where the chore manages an entire execution between when the flag is flipped and when the test thread gets back around to continuing execution. Make the test a little more pessimistic about its view of the world.

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            ndimiduk Nick Dimiduk
            ndimiduk Nick Dimiduk

            Dates

              Created:
              Updated:

              Slack

                Issue deployment