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

Add timeouts on all tests so less zombie sightings

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.0.2, 1.2.0, 1.3.0, 1.1.3, 2.0.0
    • test
    • None
    • Reviewed
    • Hide
      Use junit facility to impose timeout on test. Use test category to chose which timeout to apply: small tests timeout after 30 seconds, medium tests after 180 seconds, and large tests after ten minutes.

      Updated junit version from 4.11 to 4.12. 4.12 has support for feature used here.

      Add this at the head of your junit4 class to add a category-based timeout:

      {code}
      @Rule public final TestRule timeout = CategoryBasedTimeout.builder().withTimeout(this.getClass()).
            withLookingForStuckThread(true).build();
      {code}

      For example:

      Show
      Use junit facility to impose timeout on test. Use test category to chose which timeout to apply: small tests timeout after 30 seconds, medium tests after 180 seconds, and large tests after ten minutes. Updated junit version from 4.11 to 4.12. 4.12 has support for feature used here. Add this at the head of your junit4 class to add a category-based timeout: {code} @Rule public final TestRule timeout = CategoryBasedTimeout.builder().withTimeout(this.getClass()).       withLookingForStuckThread(true).build(); {code} For example:

    Description

      Apache9 and octo47 have been working hard at trying to get our builds passing again. They are almost there. TRUNK just failed with a zombie TestMasterObserver. Help the lads out by adding timeouts on all tests so less zombie incidence... will help identify the frequent failing issues.

      Attachments

        1. 13127.alternate.txt
          4 kB
          Michael Stack
        2. 13127.alternate.txt
          4 kB
          Nick Dimiduk
        3. 13127.alternate.txt
          4 kB
          Nick Dimiduk
        4. 13127.alternate.txt
          4 kB
          Michael Stack
        5. 13127.alternate.v2.txt
          7 kB
          Michael Stack
        6. 13127.alternate.v3.txt
          61 kB
          Michael Stack
        7. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        8. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        9. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        10. 13127.alternate.v4.txt
          68 kB
          Michael Stack
        11. 13127.alternate.v5.txt
          68 kB
          Michael Stack
        12. 13127.alternate.v6.txt
          68 kB
          Michael Stack
        13. 13127.alternate.v6.txt
          68 kB
          Michael Stack
        14. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        15. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        16. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        17. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        18. 13127.alternate.v8.txt
          67 kB
          Michael Stack
        19. 13127.alternate.v8.txt
          67 kB
          Nick Dimiduk
        20. 13127.alternative.v8.branch-1.txt
          6 kB
          Michael Stack
        21. 13127.txt
          850 kB
          Michael Stack
        22. 13127v2.txt
          830 kB
          Michael Stack

        Activity

          People

            stack Michael Stack
            stack Michael Stack
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: