Uploaded image for project: 'Hadoop Map/Reduce'
  1. Hadoop Map/Reduce
  2. MAPREDUCE-6339

Job history file is not flushed correctly because isTimerActive flag is not set true when flushTimerTask is scheduled.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 2.7.0
    • Fix Version/s: 2.8.0, 2.7.1, 3.0.0-alpha1
    • Component/s: mrv2
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      Reviewed

      Description

      Job history file is not flushed correctly because isTimerActive flag is not set true when flushTimerTask is scheduled.
      It looks like we should set isTimerActive to true when flushTimerTask is scheduled. Otherwise if a new qualified event comes before the current flush timer is expired, flushTimerTask will be canceled and rescheduled.
      Also I didn't find any code which set isTimerActive flag to true, So isTimerActive is useless in current code.

      1. MAPREDUCE-6339.000.patch
        3 kB
        zhihai xu
      2. MAPREDUCE-6339.test1.patch
        1.0 kB
        zhihai xu
      3. MAPREDUCE-6339.test.patch
        1 kB
        zhihai xu

        Issue Links

          Activity

          Hide
          zxu zhihai xu added a comment -

          thanks Devaraj K for the review and committing the patch, I am really appreciated.

          Show
          zxu zhihai xu added a comment - thanks Devaraj K for the review and committing the patch, I am really appreciated.
          Hide
          hudson Hudson added a comment -

          SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2129 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2129/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          Show
          hudson Hudson added a comment - SUCCESS: Integrated in Hadoop-Mapreduce-trunk #2129 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2129/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #180 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/180/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #180 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/180/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk #913 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/913/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #913 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/913/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #170 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/170/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #170 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/170/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #179 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/179/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #179 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/179/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-Hdfs-trunk #2111 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2111/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2111 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2111/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Hide
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #7704 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7704/)
          MAPREDUCE-6339. Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4)

          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java
          • hadoop-mapreduce-project/CHANGES.txt
          • hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Show
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #7704 (See https://builds.apache.org/job/Hadoop-trunk-Commit/7704/ ) MAPREDUCE-6339 . Job history file is not flushed correctly because (devaraj: rev f5b38477f9d0827b238fadd260c1dd2889531fd4) hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobHistoryEventHandler.java hadoop-mapreduce-project/CHANGES.txt hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-app/src/test/java/org/apache/hadoop/mapreduce/jobhistory/TestJobHistoryEventHandler.java
          Hide
          devaraj.k Devaraj K added a comment -

          Thanks zhihai xu.

          Committed to trunk, branch-2 and branch-2.7.

          Show
          devaraj.k Devaraj K added a comment - Thanks zhihai xu . Committed to trunk, branch-2 and branch-2.7.
          Hide
          devaraj.k Devaraj K added a comment -

          Thanks zhihai xu for your effort on identifying the checkstyle issue.

          MAPREDUCE-6339.000.patch looks good to me, will commit it soon.

          Show
          devaraj.k Devaraj K added a comment - Thanks zhihai xu for your effort on identifying the checkstyle issue. MAPREDUCE-6339 .000.patch looks good to me, will commit it soon.
          Hide
          zxu zhihai xu added a comment -

          The two test patches MAPREDUCE-6339.test.patch and MAPREDUCE-6339.test1.patch is to prove this is most likely a checkstyle script issue. The only difference between MAPREDUCE-6339.test.patch and MAPREDUCE-6339.test1.patch is an import statement, which doesn't look like an issue for me.
          The original patch MAPREDUCE-6339.000.patch is the one for review and commit

          Show
          zxu zhihai xu added a comment - The two test patches MAPREDUCE-6339 .test.patch and MAPREDUCE-6339 .test1.patch is to prove this is most likely a checkstyle script issue. The only difference between MAPREDUCE-6339 .test.patch and MAPREDUCE-6339 .test1.patch is an import statement, which doesn't look like an issue for me. The original patch MAPREDUCE-6339 .000.patch is the one for review and commit
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 58s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 36s There were no new javac warning messages.
          +1 javadoc 9m 45s There were no new javadoc warning messages.
          +1 release audit 0m 21s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 4m 4s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 37s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 findbugs 0m 59s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 9m 25s Tests passed in hadoop-mapreduce-client-app.
              49m 20s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729056/MAPREDUCE-6339.test.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 8f82970
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/testReport/
          Java 1.7.0_55
          uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 58s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 36s There were no new javac warning messages. +1 javadoc 9m 45s There were no new javadoc warning messages. +1 release audit 0m 21s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 4m 4s The applied patch generated 1 additional checkstyle issues. +1 install 1m 37s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 59s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 9m 25s Tests passed in hadoop-mapreduce-client-app.     49m 20s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729056/MAPREDUCE-6339.test.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 8f82970 checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/testReport/ Java 1.7.0_55 uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5473/console This message was automatically generated.
          Hide
          zxu zhihai xu added a comment -

          The last Jenkins run didn't have checkstyle issue. The only difference between these two patches is an import statement:

          import com.google.common.annotations.VisibleForTesting;
          

          It looks like the import statement causes this checkstyle issue. But this import statement looks good to me.
          Don't know why the checkstyle script report this issue. So it looks like a script issue.

          Show
          zxu zhihai xu added a comment - The last Jenkins run didn't have checkstyle issue. The only difference between these two patches is an import statement: import com.google.common.annotations.VisibleForTesting; It looks like the import statement causes this checkstyle issue. But this import statement looks good to me. Don't know why the checkstyle script report this issue. So it looks like a script issue.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 35s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 29s There were no new javac warning messages.
          +1 javadoc 9m 37s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          +1 checkstyle 5m 28s There were no new checkstyle issues.
          +1 install 1m 33s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 0m 55s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 9m 43s Tests passed in hadoop-mapreduce-client-app.
              50m 18s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12729003/MAPREDUCE-6339.test1.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 439614b
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/testReport/
          Java 1.7.0_55
          uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 35s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 29s There were no new javac warning messages. +1 javadoc 9m 37s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 5m 28s There were no new checkstyle issues. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 0m 55s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 9m 43s Tests passed in hadoop-mapreduce-client-app.     50m 18s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12729003/MAPREDUCE-6339.test1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 439614b hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/testReport/ Java 1.7.0_55 uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5471/console This message was automatically generated.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 51s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 35s There were no new javac warning messages.
          +1 javadoc 9m 38s There were no new javadoc warning messages.
          +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 5m 19s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 35s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 8m 43s Tests passed in hadoop-mapreduce-client-app.
              49m 37s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728944/MAPREDUCE-6339.001.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 5190923
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/testReport/
          Java 1.7.0_55
          uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 51s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 35s There were no new javac warning messages. +1 javadoc 9m 38s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 5m 19s The applied patch generated 1 additional checkstyle issues. +1 install 1m 35s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 8m 43s Tests passed in hadoop-mapreduce-client-app.     49m 37s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728944/MAPREDUCE-6339.001.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 5190923 checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/testReport/ Java 1.7.0_55 uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5469/console This message was automatically generated.
          Hide
          zxu zhihai xu added a comment -

          Hi Devaraj K, Just let you know the second patch MAPREDUCE-6339.001.patch is used to test checkstyle.
          The first patch MAPREDUCE-6339.000.patch is the one for review and commit. I didn't find any code style issue in my first patch MAPREDUCE-6339.000.patch. It looks like something wrong in the checkstyle script.

          Show
          zxu zhihai xu added a comment - Hi Devaraj K , Just let you know the second patch MAPREDUCE-6339 .001.patch is used to test checkstyle. The first patch MAPREDUCE-6339 .000.patch is the one for review and commit. I didn't find any code style issue in my first patch MAPREDUCE-6339 .000.patch. It looks like something wrong in the checkstyle script.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 15m 5s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 48s There were no new javac warning messages.
          +1 javadoc 9m 58s There were no new javadoc warning messages.
          +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 5m 35s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 36s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 findbugs 0m 56s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 9m 25s Tests passed in hadoop-mapreduce-client-app.
              51m 22s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728911/MAPREDUCE-6339.001.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / bc1bd7e
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/testReport/
          Java 1.7.0_55
          uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 15m 5s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 48s There were no new javac warning messages. +1 javadoc 9m 58s There were no new javadoc warning messages. +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 5m 35s The applied patch generated 1 additional checkstyle issues. +1 install 1m 36s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 56s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 9m 25s Tests passed in hadoop-mapreduce-client-app.     51m 22s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728911/MAPREDUCE-6339.001.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / bc1bd7e checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/testReport/ Java 1.7.0_55 uname Linux asf904.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5467/console This message was automatically generated.
          Hide
          zxu zhihai xu added a comment -

          I did several tests with different patches. The behavior from checkstyle is strange. It even reports the checkstyle issues from the file I didn't touch. So It looks like my first patch MAPREDUCE-6339.000.patch didn't introduce any new checkstyle issues.

          Show
          zxu zhihai xu added a comment - I did several tests with different patches. The behavior from checkstyle is strange. It even reports the checkstyle issues from the file I didn't touch. So It looks like my first patch MAPREDUCE-6339 .000.patch didn't introduce any new checkstyle issues.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 50s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace.
          -1 javac 7m 46s The applied patch generated 3 additional warning messages.
          +1 javadoc 9m 46s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 4m 4s The applied patch generated 6 additional checkstyle issues.
          +1 install 1m 36s mvn install still works.
          +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
          +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          -1 mapreduce tests 9m 14s Tests failed in hadoop-mapreduce-client-app.
              49m 10s  



          Reason Tests
          Failed unit tests hadoop.mapreduce.v2.app.webapp.TestAMWebServicesTasks
            hadoop.mapreduce.v2.app.webapp.TestAMWebApp



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728892/MAPREDUCE-6339.001.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / eccf709
          javac https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/diffJavacWarnings.txt
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/testReport/
          Java 1.7.0_55
          uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 50s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. -1 tests included 0m 0s 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 whitespace 0m 0s The patch has no lines that end in whitespace. -1 javac 7m 46s The applied patch generated 3 additional warning messages. +1 javadoc 9m 46s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 4m 4s The applied patch generated 6 additional checkstyle issues. +1 install 1m 36s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings. -1 mapreduce tests 9m 14s Tests failed in hadoop-mapreduce-client-app.     49m 10s   Reason Tests Failed unit tests hadoop.mapreduce.v2.app.webapp.TestAMWebServicesTasks   hadoop.mapreduce.v2.app.webapp.TestAMWebApp Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728892/MAPREDUCE-6339.001.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / eccf709 javac https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/diffJavacWarnings.txt checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/testReport/ Java 1.7.0_55 uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5466/console This message was automatically generated.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 53s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 34s There were no new javac warning messages.
          +1 javadoc 9m 34s There were no new javadoc warning messages.
          +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 4m 2s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 33s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 findbugs 0m 58s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          -1 mapreduce tests 9m 11s Tests failed in hadoop-mapreduce-client-app.
              48m 49s  



          Reason Tests
          Failed unit tests hadoop.mapreduce.v2.app.webapp.TestAMWebServices



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728852/MAPREDUCE-6339.001.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / eccf709
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/testReport/
          Java 1.7.0_55
          uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 53s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 34s There were no new javac warning messages. +1 javadoc 9m 34s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 4m 2s The applied patch generated 1 additional checkstyle issues. +1 install 1m 33s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 58s The patch does not introduce any new Findbugs (version 2.0.3) warnings. -1 mapreduce tests 9m 11s Tests failed in hadoop-mapreduce-client-app.     48m 49s   Reason Tests Failed unit tests hadoop.mapreduce.v2.app.webapp.TestAMWebServices Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728852/MAPREDUCE-6339.001.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / eccf709 checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/testReport/ Java 1.7.0_55 uname Linux asf903.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5464/console This message was automatically generated.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 30s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 30s There were no new javac warning messages.
          +1 javadoc 9m 37s There were no new javadoc warning messages.
          +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 3m 55s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 35s mvn install still works.
          +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse.
          +1 findbugs 0m 56s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 10m 54s Tests passed in hadoop-mapreduce-client-app.
              49m 56s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728719/MAPREDUCE-6339.001.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / feb68cb
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/testReport/
          Java 1.7.0_55
          uname Linux asf901.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 30s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 30s There were no new javac warning messages. +1 javadoc 9m 37s There were no new javadoc warning messages. +1 release audit 0m 22s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 3m 55s The applied patch generated 1 additional checkstyle issues. +1 install 1m 35s mvn install still works. +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse. +1 findbugs 0m 56s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 10m 54s Tests passed in hadoop-mapreduce-client-app.     49m 56s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728719/MAPREDUCE-6339.001.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / feb68cb checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/testReport/ Java 1.7.0_55 uname Linux asf901.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5463/console This message was automatically generated.
          Hide
          zxu zhihai xu added a comment -

          thanks Devaraj K for the review, I am not sure what cause the checkstyle issue. I uploaded a new patch, which fix 80-column-limit violations at line 259 of JobHistoryEventHandler.java.

          Show
          zxu zhihai xu added a comment - thanks Devaraj K for the review, I am not sure what cause the checkstyle issue. I uploaded a new patch, which fix 80-column-limit violations at line 259 of JobHistoryEventHandler.java.
          Hide
          devaraj.k Devaraj K added a comment -

          zhihai xu, Can you look into this checkstyle issue in the patch?

          -1	checkstyle	3m 58s	The applied patch generated 1 additional checkstyle issues.
          
          Show
          devaraj.k Devaraj K added a comment - zhihai xu , Can you look into this checkstyle issue in the patch? -1 checkstyle 3m 58s The applied patch generated 1 additional checkstyle issues.
          Hide
          zxu zhihai xu added a comment -

          Change it to a critical issue because we saw a MR job whose AM container is killed and two hours event records are missing in the job history file.

          Show
          zxu zhihai xu added a comment - Change it to a critical issue because we saw a MR job whose AM container is killed and two hours event records are missing in the job history file.
          Hide
          hadoopqa Hadoop QA added a comment -



          -1 overall



          Vote Subsystem Runtime Comment
          0 pre-patch 14m 38s Pre-patch trunk compilation is healthy.
          +1 @author 0m 0s The patch does not contain any @author tags.
          +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
          +1 whitespace 0m 0s The patch has no lines that end in whitespace.
          +1 javac 7m 30s There were no new javac warning messages.
          +1 javadoc 9m 31s There were no new javadoc warning messages.
          +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
          -1 checkstyle 3m 58s The applied patch generated 1 additional checkstyle issues.
          +1 install 1m 34s mvn install still works.
          +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse.
          +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings.
          +1 mapreduce tests 10m 2s Tests passed in hadoop-mapreduce-client-app.
              49m 13s  



          Subsystem Report/Notes
          Patch URL http://issues.apache.org/jira/secure/attachment/12728306/MAPREDUCE-6339.000.patch
          Optional Tests javadoc javac unit findbugs checkstyle
          git revision trunk / 5e67c4d
          checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/artifact/patchprocess/checkstyle-result-diff.txt
          hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt
          Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/testReport/
          Java 1.7.0_55
          uname Linux asf901.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
          Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/console

          This message was automatically generated.

          Show
          hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 14m 38s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 whitespace 0m 0s The patch has no lines that end in whitespace. +1 javac 7m 30s There were no new javac warning messages. +1 javadoc 9m 31s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 3m 58s The applied patch generated 1 additional checkstyle issues. +1 install 1m 34s mvn install still works. +1 eclipse:eclipse 0m 32s The patch built with eclipse:eclipse. +1 findbugs 0m 57s The patch does not introduce any new Findbugs (version 2.0.3) warnings. +1 mapreduce tests 10m 2s Tests passed in hadoop-mapreduce-client-app.     49m 13s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12728306/MAPREDUCE-6339.000.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / 5e67c4d checkstyle https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/artifact/patchprocess/checkstyle-result-diff.txt hadoop-mapreduce-client-app test log https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/artifact/patchprocess/testrun_hadoop-mapreduce-client-app.txt Test Results https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/testReport/ Java 1.7.0_55 uname Linux asf901.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5455/console This message was automatically generated.
          Hide
          devaraj.k Devaraj K added a comment -

          Thanks for the patch zhihai xu, submitting patch for jenkins report.

          Show
          devaraj.k Devaraj K added a comment - Thanks for the patch zhihai xu , submitting patch for jenkins report.

            People

            • Assignee:
              zxu zhihai xu
              Reporter:
              zxu zhihai xu
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development