Hadoop Common
  1. Hadoop Common
  2. HADOOP-5457

Failing contrib tests should not stop the build

    Details

    • Type: Test Test
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.20.0
    • Fix Version/s: 0.21.0
    • Component/s: test
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    • Release Note:
      Fixed the build to make sure that all the unit tests in contrib are run, regardless of the success/failure status of the previous projects' tests.

      Description

      If one or more of the unit tests in HdfsProxy fail, none of the subsequent test suites are run. All of the unit tests should run, regardless of previous projects' status.

      1. Hadoop-5457-y20.patch
        2 kB
        Hemanth Yamijala
      2. Hadoop-5457.patch
        3 kB
        Giridharan Kesavan

        Issue Links

          Activity

          Hide
          Chris Douglas added a comment -

          [ edit - Removing Chukwa from description, since it is no longer in contrib ]

          Show
          Chris Douglas added a comment - [ edit - Removing Chukwa from description, since it is no longer in contrib ]
          Hide
          Hemanth Yamijala added a comment -

          Seems a build issue. Same thing happened yesterday when I was testing HADOOP-5828. Streaming tests were failing (for some time now on trunk) and it wasn't executing any other contrib tests. Needed to remove the offending tests to run everything again.

          Show
          Hemanth Yamijala added a comment - Seems a build issue. Same thing happened yesterday when I was testing HADOOP-5828 . Streaming tests were failing (for some time now on trunk) and it wasn't executing any other contrib tests. Needed to remove the offending tests to run everything again.
          Hide
          Giridharan Kesavan added a comment -

          this patch fixes this issue and would run all the contrib unit tests.
          tnx

          Show
          Giridharan Kesavan added a comment - this patch fixes this issue and would run all the contrib unit tests. tnx
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12409873/Hadoop-5457.patch
          against trunk revision 781739.

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

          +1 tests included. The patch appears to include 4 new or modified tests.

          +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 warnings.

          +1 Eclipse classpath. The patch retains Eclipse classpath integrity.

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

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

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

          Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/testReport/
          Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/artifact/trunk/build/test/checkstyle-errors.html
          Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/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/12409873/Hadoop-5457.patch against trunk revision 781739. +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 4 new or modified tests. +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 warnings. +1 Eclipse classpath. The patch retains Eclipse classpath integrity. +1 release audit. The applied patch does not increase the total number of release audit warnings. -1 core tests. The patch failed core unit tests. -1 contrib tests. The patch failed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-vesta.apache.org/462/console This message is automatically generated.
          Hide
          Giridharan Kesavan added a comment -

          I tested test-contrib locally, it ran contrib tests for all the contrib components though "org.apache.hadoop.mapred.TestQueueCapacities" FAILED timeout.

          Show
          Giridharan Kesavan added a comment - I tested test-contrib locally, it ran contrib tests for all the contrib components though "org.apache.hadoop.mapred.TestQueueCapacities" FAILED timeout.
          Hide
          Sharad Agarwal added a comment -

          +1 patch looks fine.

          Show
          Sharad Agarwal added a comment - +1 patch looks fine.
          Hide
          Nigel Daley added a comment -

          +1. Looks good.

          Show
          Nigel Daley added a comment - +1. Looks good.
          Hide
          Hemanth Yamijala added a comment -

          Patch for earlier version of Hadoop. Not for commit here.

          Show
          Hemanth Yamijala added a comment - Patch for earlier version of Hadoop. Not for commit here.
          Hide
          Konstantin Boudnik added a comment -

          It seems to me that the problem still exists and is fully addressed by the patch in HADOOP-6506

          Show
          Konstantin Boudnik added a comment - It seems to me that the problem still exists and is fully addressed by the patch in HADOOP-6506

            People

            • Assignee:
              Giridharan Kesavan
              Reporter:
              Chris Douglas
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development