Hadoop Map/Reduce
  1. Hadoop Map/Reduce
  2. MAPREDUCE-2367

Allow using a file to exclude certain tests from build

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.23.0
    • Fix Version/s: 0.23.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      It would be nice to be able to exclude certain tests when running builds. For example, when a test is "known flaky", you may want to exclude it from the main Hudson job, but not actually disable it in the codebase (so that it still runs as part of another Hudson job, for example).

      1. mapreduce-2367-cygwin-fix.txt
        1 kB
        Todd Lipcon
      2. mapreduce-2367.txt
        1 kB
        Todd Lipcon

        Issue Links

          Activity

          Todd Lipcon created issue -
          Todd Lipcon made changes -
          Field Original Value New Value
          Link This issue is a clone of HADOOP-7167 [ HADOOP-7167 ]
          Todd Lipcon made changes -
          Project Hadoop Common [ 12310240 ] Hadoop Map/Reduce [ 12310941 ]
          Key HADOOP-7168 MAPREDUCE-2367
          Affects Version/s 0.23.0 [ 12315570 ]
          Affects Version/s 0.23.0 [ 12315569 ]
          Fix Version/s 0.23.0 [ 12315570 ]
          Fix Version/s 0.23.0 [ 12315569 ]
          Todd Lipcon made changes -
          Attachment mapreduce-2367.txt [ 12472872 ]
          Todd Lipcon made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Konstantin Boudnik added a comment -

          +1 patch looks good. With only exception that we shouldn't overuse this 'facility'

          Show
          Konstantin Boudnik added a comment - +1 patch looks good. With only exception that we shouldn't overuse this 'facility'
          Hide
          Todd Lipcon added a comment -

          Committed. Thanks Cos.

          Show
          Todd Lipcon added a comment - Committed. Thanks Cos.
          Todd Lipcon made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Hadoop Flags [Reviewed]
          Resolution Fixed [ 1 ]
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12472872/mapreduce-2367.txt
          against trunk revision 1078964.

          +1 @author. The patch does not contain any @author tags.

          -1 tests included. The patch doesn't appear to include any new or modified tests.
          Please justify why no new tests are needed for this patch.
          Also please list what manual steps were performed to verify this patch.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed core unit tests.

          -1 contrib tests. The patch failed contrib unit tests.

          +1 system test framework. The patch passed system test framework compile.

          Test results: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//testReport/
          Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12472872/mapreduce-2367.txt against trunk revision 1078964. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. -1 contrib tests. The patch failed contrib unit tests. +1 system test framework. The patch passed system test framework compile. Test results: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//testReport/ Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://hudson.apache.org/hudson/job/PreCommit-MAPREDUCE-Build/124//console This message is automatically generated.
          Hide
          Todd Lipcon added a comment -

          Reopening to fix for cygwin

          Show
          Todd Lipcon added a comment - Reopening to fix for cygwin
          Todd Lipcon made changes -
          Resolution Fixed [ 1 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          Hide
          Todd Lipcon added a comment -

          Fix for cygwin

          Show
          Todd Lipcon added a comment - Fix for cygwin
          Todd Lipcon made changes -
          Attachment mapreduce-2367-cygwin-fix.txt [ 12473455 ]
          Hide
          Tsz Wo Nicholas Sze added a comment -

          +1

          Show
          Tsz Wo Nicholas Sze added a comment - +1
          Hide
          Todd Lipcon added a comment -

          Committed fix to trunk

          Show
          Todd Lipcon added a comment - Committed fix to trunk
          Todd Lipcon made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #631 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/631/)
          MAPREDUCE-2367. Amend previous commit for this JIRA to fix build on Cygwin. Contributed by Todd Lipcon.

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #631 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/631/ ) MAPREDUCE-2367 . Amend previous commit for this JIRA to fix build on Cygwin. Contributed by Todd Lipcon.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk-Commit #636 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/636/)
          Remove redundant entry for MAPREDUCE-2367 in CHANGES.txt

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #636 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/636/ ) Remove redundant entry for MAPREDUCE-2367 in CHANGES.txt
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/)

          Show
          Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #643 (See https://hudson.apache.org/hudson/job/Hadoop-Mapreduce-trunk/643/ )
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Patch Available Patch Available
          53s 1 Todd Lipcon 07/Mar/11 22:49
          Patch Available Patch Available Resolved Resolved
          2h 46m 1 Todd Lipcon 08/Mar/11 01:35
          Resolved Resolved Reopened Reopened
          3d 22h 51m 1 Todd Lipcon 12/Mar/11 00:27
          Reopened Reopened Resolved Resolved
          31m 28s 1 Todd Lipcon 12/Mar/11 00:58
          Resolved Resolved Closed Closed
          247d 23h 50m 1 Arun C Murthy 15/Nov/11 00:49

            People

            • Assignee:
              Todd Lipcon
              Reporter:
              Todd Lipcon
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development