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. 13127v2.txt
          830 kB
          Michael Stack
        2. 13127.txt
          850 kB
          Michael Stack
        3. 13127.alternative.v8.branch-1.txt
          6 kB
          Michael Stack
        4. 13127.alternate.v8.txt
          67 kB
          Nick Dimiduk
        5. 13127.alternate.v8.txt
          67 kB
          Michael Stack
        6. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        7. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        8. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        9. 13127.alternate.v7.txt
          66 kB
          Michael Stack
        10. 13127.alternate.v6.txt
          68 kB
          Michael Stack
        11. 13127.alternate.v6.txt
          68 kB
          Michael Stack
        12. 13127.alternate.v5.txt
          68 kB
          Michael Stack
        13. 13127.alternate.v4.txt
          68 kB
          Michael Stack
        14. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        15. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        16. 13127.alternate.v3.txt
          7 kB
          Michael Stack
        17. 13127.alternate.v3.txt
          61 kB
          Michael Stack
        18. 13127.alternate.v2.txt
          7 kB
          Michael Stack
        19. 13127.alternate.txt
          4 kB
          Michael Stack
        20. 13127.alternate.txt
          4 kB
          Nick Dimiduk
        21. 13127.alternate.txt
          4 kB
          Nick Dimiduk
        22. 13127.alternate.txt
          4 kB
          Michael Stack
        There are no Sub-Tasks for this issue.

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: