Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-19948 Since HBASE-19873, HBaseClassTestRule, Small/Medium/Large has different semantic
  3. HBASE-19956

Remove category as a consideration timing out tests; set all test to timeout at 10minutes regardless

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.0-beta-2, 2.0.0
    • None
    • None

    Description

      Appy suggestion from parent issue.

      Notion is that during transition measuring timeout on a class-basis instead of on a test method basis, categories only mess us up. The only important regard when it comes to timeout is that our timeout cuts in before the surefire one does; it just does a savage kill w/o regard for who is about. If we kill, there'll be info on culprit. Thats what we want.

      We used to have timeout per test method. It was changed for hbase2 to be class based. Easier for devs to grok. For now, we purge changing test class timeout based off category and just paint all tests w/ the one 10-minute max brush.

      Attachments

        1. HBASE-19956.branch-2.001.patch
          2 kB
          Michael Stack

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: