Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-4904 YARN Log tooling enhancements
  3. YARN-6011

Add a new web service to list the files on a container in AHSWebService

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.9.0, 3.0.0-alpha2
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed
    1. YARN-6011.1.patch
      20 kB
      Xuan Gong
    2. YARN-6011.2.patch
      21 kB
      Xuan Gong
    3. YARN-6011.3.patch
      17 kB
      Xuan Gong
    4. YARN-6011.4.patch
      21 kB
      Xuan Gong

      Issue Links

        Activity

        Hide
        djp Junping Du added a comment -

        I have commit the patch to trunk and branch-2. Thanks Xuan Gong for patch contribution!

        Show
        djp Junping Du added a comment - I have commit the patch to trunk and branch-2. Thanks Xuan Gong for patch contribution!
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11122 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11122/)
        YARN-6011. Add a new web service to list the files on a container in (junping_du: rev cf695577aa6d9715b77ab7309ecd792ef226c439)

        • (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/java/org/apache/hadoop/yarn/server/webapp/dao/ContainerLogsInfo.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/webapp/AHSWebServices.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/webapp/TestAHSWebServices.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #11122 (See https://builds.apache.org/job/Hadoop-trunk-Commit/11122/ ) YARN-6011 . Add a new web service to list the files on a container in (junping_du: rev cf695577aa6d9715b77ab7309ecd792ef226c439) (add) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/java/org/apache/hadoop/yarn/server/webapp/dao/ContainerLogsInfo.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/main/java/org/apache/hadoop/yarn/server/applicationhistoryservice/webapp/AHSWebServices.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice/src/test/java/org/apache/hadoop/yarn/server/applicationhistoryservice/webapp/TestAHSWebServices.java
        Hide
        djp Junping Du added a comment -

        +1.

        Show
        djp Junping Du added a comment - +1.
        Hide
        xgong Xuan Gong added a comment -

        Junping Du The testcase failure is not related.

        Show
        xgong Xuan Gong added a comment - Junping Du The testcase failure is not related.
        Hide
        djp Junping Du added a comment -

        Latest patch LGTM. However, Xuan Gong, can you check if test failure is related to you patch?

        Show
        djp Junping Du added a comment - Latest patch LGTM. However, Xuan Gong , can you check if test failure is related to you patch?
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 20s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
        0 mvndep 0m 10s Maven dependency ordering for branch
        +1 mvninstall 14m 35s trunk passed
        +1 compile 1m 50s trunk passed
        +1 checkstyle 0m 35s trunk passed
        +1 mvnsite 0m 48s trunk passed
        +1 mvneclipse 0m 30s trunk passed
        +1 findbugs 1m 26s trunk passed
        +1 javadoc 0m 33s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 0m 41s the patch passed
        +1 compile 1m 50s the patch passed
        +1 javac 1m 50s the patch passed
        -0 checkstyle 0m 33s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 2 new + 30 unchanged - 1 fixed = 32 total (was 31)
        +1 mvnsite 0m 46s the patch passed
        +1 mvneclipse 0m 27s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 44s the patch passed
        +1 javadoc 0m 32s the patch passed
        +1 unit 0m 30s hadoop-yarn-server-common in the patch passed.
        -1 unit 2m 57s hadoop-yarn-server-applicationhistoryservice in the patch failed.
        +1 asflicense 0m 20s The patch does not generate ASF License warnings.
        36m 39s



        Reason Tests
        Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-6011
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12846685/YARN-6011.4.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux 8c65d5f99881 3.13.0-105-generic #152-Ubuntu SMP Fri Dec 2 15:37:11 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / e692316
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14629/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14629/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14629/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14629/console
        Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 20s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 10s Maven dependency ordering for branch +1 mvninstall 14m 35s trunk passed +1 compile 1m 50s trunk passed +1 checkstyle 0m 35s trunk passed +1 mvnsite 0m 48s trunk passed +1 mvneclipse 0m 30s trunk passed +1 findbugs 1m 26s trunk passed +1 javadoc 0m 33s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 0m 41s the patch passed +1 compile 1m 50s the patch passed +1 javac 1m 50s the patch passed -0 checkstyle 0m 33s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 2 new + 30 unchanged - 1 fixed = 32 total (was 31) +1 mvnsite 0m 46s the patch passed +1 mvneclipse 0m 27s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 44s the patch passed +1 javadoc 0m 32s the patch passed +1 unit 0m 30s hadoop-yarn-server-common in the patch passed. -1 unit 2m 57s hadoop-yarn-server-applicationhistoryservice in the patch failed. +1 asflicense 0m 20s The patch does not generate ASF License warnings. 36m 39s Reason Tests Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6011 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12846685/YARN-6011.4.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 8c65d5f99881 3.13.0-105-generic #152-Ubuntu SMP Fri Dec 2 15:37:11 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / e692316 Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14629/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14629/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14629/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server Console output https://builds.apache.org/job/PreCommit-YARN-Build/14629/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        xgong Xuan Gong added a comment -

        Thanks for the review. Junping Du

        1. For generating URI embed in response of getContainerLogsInfo, I saw some very similar one in other places, like: getLogs(). Can we refactor the code a bit to reuse the same logic?

        Yes, other than this, we have lots of duplicate codes related to webservice. Create a separate jira for the refactory work: https://issues.apache.org/jira/browse/YARN-6080

        Also added the TODO for this.

        2. For getContainerLogsInfo(), if an app is not in running or finished state, here we will return bad request. However, I remember in our ATS implementation, RM after restart could send regressioned application state event to ATS, like app creation event to ATS which was running before. Can you double check ATS's app status won't have regression? Otherwise, we shouldn't just simply return a bad request.

        This is fine. Based on how we generate ApplicationReport from ATS entities, the CREATE event will only provide the created time for the application. The application state is generated based on ApplicationMetricsConstants.STATE_EVENT_INFO which is provided by ApplicationFinishedEvent and ApplicationStateUpdatedEvent. For the ApplicationStateUpdatedEvent, we only submit the event when the app transits from ACCEPTED to RUNNING state.

        So, when the Application is in RUNNING state in RM, the state will be RUNNING in ATS. Event if the RM restart/ AM restart happens later, the state will not be changed. So, I think that right now, it is fine that we only check for the RUNNING state here .

        3. For getContainerLogMeta(), I remember I have some previous comments on refactor code (consolidate similar logic, especially log reader) in previous JIRAs. How's going with that effort? If that effort is not a short term priory for you, please add a TODO here - may be someone else read this part of code could help on that.

        Link the jira: https://issues.apache.org/jira/browse/YARN-4993

        Added the TODO for this.

        Show
        xgong Xuan Gong added a comment - Thanks for the review. Junping Du 1. For generating URI embed in response of getContainerLogsInfo, I saw some very similar one in other places, like: getLogs(). Can we refactor the code a bit to reuse the same logic? Yes, other than this, we have lots of duplicate codes related to webservice. Create a separate jira for the refactory work: https://issues.apache.org/jira/browse/YARN-6080 Also added the TODO for this. 2. For getContainerLogsInfo(), if an app is not in running or finished state, here we will return bad request. However, I remember in our ATS implementation, RM after restart could send regressioned application state event to ATS, like app creation event to ATS which was running before. Can you double check ATS's app status won't have regression? Otherwise, we shouldn't just simply return a bad request. This is fine. Based on how we generate ApplicationReport from ATS entities, the CREATE event will only provide the created time for the application. The application state is generated based on ApplicationMetricsConstants.STATE_EVENT_INFO which is provided by ApplicationFinishedEvent and ApplicationStateUpdatedEvent. For the ApplicationStateUpdatedEvent, we only submit the event when the app transits from ACCEPTED to RUNNING state. So, when the Application is in RUNNING state in RM, the state will be RUNNING in ATS. Event if the RM restart/ AM restart happens later, the state will not be changed. So, I think that right now, it is fine that we only check for the RUNNING state here . 3. For getContainerLogMeta(), I remember I have some previous comments on refactor code (consolidate similar logic, especially log reader) in previous JIRAs. How's going with that effort? If that effort is not a short term priory for you, please add a TODO here - may be someone else read this part of code could help on that. Link the jira: https://issues.apache.org/jira/browse/YARN-4993 Added the TODO for this.
        Hide
        djp Junping Du added a comment -

        Thanks Xuan Gong for delivering the patch. A couple of comments:
        1. For generating URI embed in response of getContainerLogsInfo, I saw some very similar one in other places, like: getLogs(). Can we refactor the code a bit to reuse the same logic?

        2. For getContainerLogsInfo(), if an app is not in running or finished state, here we will return bad request. However, I remember in our ATS implementation, RM after restart could send regressioned application state event to ATS, like app creation event to ATS which was running before. Can you double check ATS's app status won't have regression? Otherwise, we shouldn't just simply return a bad request.

        3. For getContainerLogMeta(), I remember I have some previous comments on refactor code (consolidate similar logic, especially log reader) in previous JIRAs. How's going with that effort? If that effort is not a short term priory for you, please add a TODO here - may be someone else read this part of code could help on that.

        Other looks good to me.

        Show
        djp Junping Du added a comment - Thanks Xuan Gong for delivering the patch. A couple of comments: 1. For generating URI embed in response of getContainerLogsInfo, I saw some very similar one in other places, like: getLogs(). Can we refactor the code a bit to reuse the same logic? 2. For getContainerLogsInfo(), if an app is not in running or finished state, here we will return bad request. However, I remember in our ATS implementation, RM after restart could send regressioned application state event to ATS, like app creation event to ATS which was running before. Can you double check ATS's app status won't have regression? Otherwise, we shouldn't just simply return a bad request. 3. For getContainerLogMeta(), I remember I have some previous comments on refactor code (consolidate similar logic, especially log reader) in previous JIRAs. How's going with that effort? If that effort is not a short term priory for you, please add a TODO here - may be someone else read this part of code could help on that. Other looks good to me.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 17s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
        0 mvndep 0m 9s Maven dependency ordering for branch
        +1 mvninstall 14m 20s trunk passed
        +1 compile 1m 55s trunk passed
        +1 checkstyle 0m 33s trunk passed
        +1 mvnsite 0m 49s trunk passed
        +1 mvneclipse 0m 29s trunk passed
        +1 findbugs 1m 16s trunk passed
        +1 javadoc 0m 30s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 0m 36s the patch passed
        +1 compile 1m 45s the patch passed
        +1 javac 1m 45s the patch passed
        -0 checkstyle 0m 32s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 2 new + 30 unchanged - 1 fixed = 32 total (was 31)
        +1 mvnsite 0m 41s the patch passed
        +1 mvneclipse 0m 26s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 36s the patch passed
        +1 javadoc 0m 32s the patch passed
        +1 unit 0m 31s hadoop-yarn-server-common in the patch passed.
        -1 unit 2m 52s hadoop-yarn-server-applicationhistoryservice in the patch failed.
        +1 asflicense 0m 18s The patch does not generate ASF License warnings.
        35m 28s



        Reason Tests
        Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-6011
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12843937/YARN-6011.2.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux 1ca4afdb2648 3.13.0-103-generic #150-Ubuntu SMP Thu Nov 24 10:34:17 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / ef2dd7b
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14365/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14365/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14365/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14365/console
        Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 17s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 9s Maven dependency ordering for branch +1 mvninstall 14m 20s trunk passed +1 compile 1m 55s trunk passed +1 checkstyle 0m 33s trunk passed +1 mvnsite 0m 49s trunk passed +1 mvneclipse 0m 29s trunk passed +1 findbugs 1m 16s trunk passed +1 javadoc 0m 30s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 0m 36s the patch passed +1 compile 1m 45s the patch passed +1 javac 1m 45s the patch passed -0 checkstyle 0m 32s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 2 new + 30 unchanged - 1 fixed = 32 total (was 31) +1 mvnsite 0m 41s the patch passed +1 mvneclipse 0m 26s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 36s the patch passed +1 javadoc 0m 32s the patch passed +1 unit 0m 31s hadoop-yarn-server-common in the patch passed. -1 unit 2m 52s hadoop-yarn-server-applicationhistoryservice in the patch failed. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 35m 28s Reason Tests Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6011 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12843937/YARN-6011.2.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 1ca4afdb2648 3.13.0-103-generic #150-Ubuntu SMP Thu Nov 24 10:34:17 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / ef2dd7b Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14365/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14365/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14365/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server Console output https://builds.apache.org/job/PreCommit-YARN-Build/14365/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 16s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 1 new or modified test files.
        0 mvndep 0m 25s Maven dependency ordering for branch
        +1 mvninstall 12m 37s trunk passed
        +1 compile 1m 32s trunk passed
        +1 checkstyle 0m 31s trunk passed
        +1 mvnsite 0m 41s trunk passed
        +1 mvneclipse 0m 27s trunk passed
        +1 findbugs 1m 8s trunk passed
        +1 javadoc 0m 30s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 0m 35s the patch passed
        +1 compile 1m 30s the patch passed
        +1 javac 1m 30s the patch passed
        -0 checkstyle 0m 29s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 7 new + 30 unchanged - 1 fixed = 37 total (was 31)
        +1 mvnsite 0m 37s the patch passed
        +1 mvneclipse 0m 23s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 1m 19s the patch passed
        +1 javadoc 0m 27s the patch passed
        +1 unit 0m 27s hadoop-yarn-server-common in the patch passed.
        -1 unit 3m 13s hadoop-yarn-server-applicationhistoryservice in the patch failed.
        +1 asflicense 0m 17s The patch does not generate ASF License warnings.
        32m 49s



        Reason Tests
        Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:a9ad5d6
        JIRA Issue YARN-6011
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12843775/YARN-6011.1.patch
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
        uname Linux b573db68b305 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / fcbe152
        Default Java 1.8.0_111
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14359/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/14359/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14359/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/14359/console
        Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 reexec 0m 16s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 1 new or modified test files. 0 mvndep 0m 25s Maven dependency ordering for branch +1 mvninstall 12m 37s trunk passed +1 compile 1m 32s trunk passed +1 checkstyle 0m 31s trunk passed +1 mvnsite 0m 41s trunk passed +1 mvneclipse 0m 27s trunk passed +1 findbugs 1m 8s trunk passed +1 javadoc 0m 30s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 0m 35s the patch passed +1 compile 1m 30s the patch passed +1 javac 1m 30s the patch passed -0 checkstyle 0m 29s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server: The patch generated 7 new + 30 unchanged - 1 fixed = 37 total (was 31) +1 mvnsite 0m 37s the patch passed +1 mvneclipse 0m 23s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 1m 19s the patch passed +1 javadoc 0m 27s the patch passed +1 unit 0m 27s hadoop-yarn-server-common in the patch passed. -1 unit 3m 13s hadoop-yarn-server-applicationhistoryservice in the patch failed. +1 asflicense 0m 17s The patch does not generate ASF License warnings. 32m 49s Reason Tests Failed junit tests hadoop.yarn.server.timeline.webapp.TestTimelineWebServices Subsystem Report/Notes Docker Image:yetus/hadoop:a9ad5d6 JIRA Issue YARN-6011 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12843775/YARN-6011.1.patch Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux b573db68b305 3.13.0-95-generic #142-Ubuntu SMP Fri Aug 12 17:00:09 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / fcbe152 Default Java 1.8.0_111 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/14359/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/14359/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-applicationhistoryservice.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/14359/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-applicationhistoryservice U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server Console output https://builds.apache.org/job/PreCommit-YARN-Build/14359/console Powered by Apache Yetus 0.5.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.

          People

          • Assignee:
            xgong Xuan Gong
            Reporter:
            xgong Xuan Gong
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development