Details

    • Hadoop Flags:
      Reviewed

      Description

      YARN-5211 proposes adding support for generalized priorities for reservations in the YARN ReservationSystem. This JIRA is a sub-task to track the changes needed in ApplicationClientProtocol to accomplish it. Please refer to the design doc in the parent JIRA for details.

      1. YARN-5384.v9.patch
        14 kB
        Sean Po
      2. YARN-5384.v8.patch
        12 kB
        Sean Po
      3. YARN-5384.v7.patch
        12 kB
        Sean Po
      4. YARN-5384.v6.patch
        12 kB
        Sean Po
      5. YARN-5384.v5.patch
        12 kB
        Sean Po
      6. YARN-5384.v4.patch
        16 kB
        Sean Po
      7. YARN-5384.v3.patch
        20 kB
        Sean Po
      8. YARN-5384.v2.patch
        20 kB
        Sean Po
      9. YARN-5384.v11.patch
        14 kB
        Sean Po
      10. YARN-5384.v10.patch
        14 kB
        Sean Po
      11. YARN-5384.v1.patch
        18 kB
        Sean Po

        Activity

        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10524 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10524/)
        YARN-5384. Expose priority in ReservationSystem submission APIs. (Sean (subru: rev 3a3697deab3e3397082222deb66fb613d86ff9ae)

        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/records/impl/pb/ReservationDefinitionPBImpl.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/reservation/ReservationSystemTestUtil.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/ReservationDefinitionInfo.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/records/ReservationDefinition.java
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/yarn_protos.proto
        • (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Jenkins build Hadoop-trunk-Commit #10524 (See https://builds.apache.org/job/Hadoop-trunk-Commit/10524/ ) YARN-5384 . Expose priority in ReservationSystem submission APIs. (Sean (subru: rev 3a3697deab3e3397082222deb66fb613d86ff9ae) (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/api/records/impl/pb/ReservationDefinitionPBImpl.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/reservation/ReservationSystemTestUtil.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/ReservationDefinitionInfo.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/java/org/apache/hadoop/yarn/api/records/ReservationDefinition.java (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api/src/main/proto/yarn_protos.proto (edit) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        Hide
        subru Subru Krishnan added a comment -

        Thanks Sean Po for the contribution. I just committed this to trunk/branch-2.

        Show
        subru Subru Krishnan added a comment - Thanks Sean Po for the contribution. I just committed this to trunk/branch-2.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 11s 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 1m 4s Maven dependency ordering for branch
        +1 mvninstall 7m 23s trunk passed
        +1 compile 2m 28s trunk passed
        +1 checkstyle 0m 39s trunk passed
        +1 mvnsite 1m 51s trunk passed
        +1 mvneclipse 0m 51s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 6s trunk passed
        +1 javadoc 1m 15s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 1m 29s the patch passed
        +1 compile 2m 43s the patch passed
        +1 cc 2m 43s the patch passed
        +1 javac 2m 43s the patch passed
        +1 checkstyle 0m 43s the patch passed
        +1 mvnsite 2m 0s the patch passed
        +1 mvneclipse 0m 46s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 54s the patch passed
        +1 javadoc 1m 17s the patch passed
        +1 unit 0m 26s hadoop-yarn-api in the patch passed.
        +1 unit 2m 32s hadoop-yarn-common in the patch passed.
        -1 unit 37m 25s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 unit 0m 12s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 26s The patch does not generate ASF License warnings.
        73m 56s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831153/YARN-5384.v11.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux a7da1f7bdcb3 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 / 2549ee9
        Default Java 1.8.0_101
        findbugs v3.0.0
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13261/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13261/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13261/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13261/console
        Powered by Apache Yetus 0.3.0 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 11s 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 1m 4s Maven dependency ordering for branch +1 mvninstall 7m 23s trunk passed +1 compile 2m 28s trunk passed +1 checkstyle 0m 39s trunk passed +1 mvnsite 1m 51s trunk passed +1 mvneclipse 0m 51s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 6s trunk passed +1 javadoc 1m 15s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 1m 29s the patch passed +1 compile 2m 43s the patch passed +1 cc 2m 43s the patch passed +1 javac 2m 43s the patch passed +1 checkstyle 0m 43s the patch passed +1 mvnsite 2m 0s the patch passed +1 mvneclipse 0m 46s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 54s the patch passed +1 javadoc 1m 17s the patch passed +1 unit 0m 26s hadoop-yarn-api in the patch passed. +1 unit 2m 32s hadoop-yarn-common in the patch passed. -1 unit 37m 25s hadoop-yarn-server-resourcemanager in the patch failed. +1 unit 0m 12s hadoop-yarn-site in the patch passed. +1 asflicense 0m 26s The patch does not generate ASF License warnings. 73m 56s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831153/YARN-5384.v11.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux a7da1f7bdcb3 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 / 2549ee9 Default Java 1.8.0_101 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/13261/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13261/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13261/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13261/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        Test failure is known: https://issues.apache.org/jira/browse/YARN-5043?jql=text%20~%20%22testRMAppAttemptFailuresValidityInterval%22.

        I also ran locally, and saw that the test ran successfully.

        Show
        seanpo03 Sean Po added a comment - Test failure is known: https://issues.apache.org/jira/browse/YARN-5043?jql=text%20~%20%22testRMAppAttemptFailuresValidityInterval%22 . I also ran locally, and saw that the test ran successfully.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 15s 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 7m 14s trunk passed
        +1 compile 2m 22s trunk passed
        +1 checkstyle 0m 38s trunk passed
        +1 mvnsite 1m 46s trunk passed
        +1 mvneclipse 0m 50s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 2m 57s trunk passed
        +1 javadoc 1m 15s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 1m 27s the patch passed
        +1 compile 2m 25s the patch passed
        +1 cc 2m 25s the patch passed
        +1 javac 2m 25s the patch passed
        +1 checkstyle 0m 37s the patch passed
        +1 mvnsite 1m 43s the patch passed
        +1 mvneclipse 0m 43s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 22s the patch passed
        +1 javadoc 1m 8s the patch passed
        +1 unit 0m 24s hadoop-yarn-api in the patch passed.
        +1 unit 2m 18s hadoop-yarn-common in the patch passed.
        -1 unit 35m 13s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 unit 0m 7s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 21s The patch does not generate ASF License warnings.
        68m 17s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831153/YARN-5384.v11.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 58533dcefd38 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 2ab1ef1
        Default Java 1.8.0_101
        findbugs v3.0.0
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13260/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13260/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13260/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13260/console
        Powered by Apache Yetus 0.3.0 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 15s 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 7m 14s trunk passed +1 compile 2m 22s trunk passed +1 checkstyle 0m 38s trunk passed +1 mvnsite 1m 46s trunk passed +1 mvneclipse 0m 50s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 2m 57s trunk passed +1 javadoc 1m 15s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 1m 27s the patch passed +1 compile 2m 25s the patch passed +1 cc 2m 25s the patch passed +1 javac 2m 25s the patch passed +1 checkstyle 0m 37s the patch passed +1 mvnsite 1m 43s the patch passed +1 mvneclipse 0m 43s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 22s the patch passed +1 javadoc 1m 8s the patch passed +1 unit 0m 24s hadoop-yarn-api in the patch passed. +1 unit 2m 18s hadoop-yarn-common in the patch passed. -1 unit 35m 13s hadoop-yarn-server-resourcemanager in the patch failed. +1 unit 0m 7s hadoop-yarn-site in the patch passed. +1 asflicense 0m 21s The patch does not generate ASF License warnings. 68m 17s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831153/YARN-5384.v11.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 58533dcefd38 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 2ab1ef1 Default Java 1.8.0_101 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/13260/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13260/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13260/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13260/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v11.patch removes the remaining checkstyle issue.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v11.patch removes the remaining checkstyle issue.
        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 10s Maven dependency ordering for branch
        +1 mvninstall 6m 44s trunk passed
        +1 compile 2m 18s trunk passed
        +1 checkstyle 0m 38s trunk passed
        +1 mvnsite 1m 51s trunk passed
        +1 mvneclipse 0m 53s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 4s trunk passed
        +1 javadoc 1m 17s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 1m 30s the patch passed
        +1 compile 2m 23s the patch passed
        +1 cc 2m 23s the patch passed
        +1 javac 2m 23s the patch passed
        -1 checkstyle 0m 37s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31)
        +1 mvnsite 1m 41s the patch passed
        +1 mvneclipse 0m 46s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 24s the patch passed
        +1 javadoc 1m 10s the patch passed
        +1 unit 0m 24s hadoop-yarn-api in the patch passed.
        +1 unit 2m 19s hadoop-yarn-common in the patch passed.
        +1 unit 35m 24s hadoop-yarn-server-resourcemanager in the patch passed.
        +1 unit 0m 8s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 18s The patch does not generate ASF License warnings.
        68m 19s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831056/YARN-5384.v10.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux d81b4d9b4e6c 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
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 10be459
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13255/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13255/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13255/console
        Powered by Apache Yetus 0.3.0 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 10s Maven dependency ordering for branch +1 mvninstall 6m 44s trunk passed +1 compile 2m 18s trunk passed +1 checkstyle 0m 38s trunk passed +1 mvnsite 1m 51s trunk passed +1 mvneclipse 0m 53s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 4s trunk passed +1 javadoc 1m 17s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 1m 30s the patch passed +1 compile 2m 23s the patch passed +1 cc 2m 23s the patch passed +1 javac 2m 23s the patch passed -1 checkstyle 0m 37s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31) +1 mvnsite 1m 41s the patch passed +1 mvneclipse 0m 46s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 24s the patch passed +1 javadoc 1m 10s the patch passed +1 unit 0m 24s hadoop-yarn-api in the patch passed. +1 unit 2m 19s hadoop-yarn-common in the patch passed. +1 unit 35m 24s hadoop-yarn-server-resourcemanager in the patch passed. +1 unit 0m 8s hadoop-yarn-site in the patch passed. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 68m 19s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831056/YARN-5384.v10.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux d81b4d9b4e6c 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 Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 10be459 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13255/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13255/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13255/console Powered by Apache Yetus 0.3.0 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 10s Maven dependency ordering for branch
        +1 mvninstall 8m 19s trunk passed
        +1 compile 2m 49s trunk passed
        +1 checkstyle 0m 41s trunk passed
        +1 mvnsite 2m 0s trunk passed
        +1 mvneclipse 0m 51s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 25s trunk passed
        +1 javadoc 1m 19s trunk passed
        0 mvndep 0m 11s Maven dependency ordering for patch
        +1 mvninstall 1m 44s the patch passed
        +1 compile 2m 39s the patch passed
        +1 cc 2m 39s the patch passed
        +1 javac 2m 39s the patch passed
        -1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31)
        +1 mvnsite 1m 54s the patch passed
        +1 mvneclipse 0m 47s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 4m 6s the patch passed
        +1 javadoc 1m 18s the patch passed
        +1 unit 0m 28s hadoop-yarn-api in the patch passed.
        +1 unit 2m 26s hadoop-yarn-common in the patch passed.
        -1 unit 36m 55s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 unit 0m 7s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 20s The patch does not generate ASF License warnings.
        74m 27s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831050/YARN-5384.v9.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 0633a5ca7d0a 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 10be459
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13254/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13254/console
        Powered by Apache Yetus 0.3.0 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 10s Maven dependency ordering for branch +1 mvninstall 8m 19s trunk passed +1 compile 2m 49s trunk passed +1 checkstyle 0m 41s trunk passed +1 mvnsite 2m 0s trunk passed +1 mvneclipse 0m 51s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 25s trunk passed +1 javadoc 1m 19s trunk passed 0 mvndep 0m 11s Maven dependency ordering for patch +1 mvninstall 1m 44s the patch passed +1 compile 2m 39s the patch passed +1 cc 2m 39s the patch passed +1 javac 2m 39s the patch passed -1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31) +1 mvnsite 1m 54s the patch passed +1 mvneclipse 0m 47s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 4m 6s the patch passed +1 javadoc 1m 18s the patch passed +1 unit 0m 28s hadoop-yarn-api in the patch passed. +1 unit 2m 26s hadoop-yarn-common in the patch passed. -1 unit 36m 55s hadoop-yarn-server-resourcemanager in the patch failed. +1 unit 0m 7s hadoop-yarn-site in the patch passed. +1 asflicense 0m 20s The patch does not generate ASF License warnings. 74m 27s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.applicationsmanager.TestAMRestart Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831050/YARN-5384.v9.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 0633a5ca7d0a 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 10be459 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13254/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13254/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13254/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment - - edited

        YARN-5384.v10.patch replaces all occurrences of the default Priority.newInstance(0) with Priority.UNDEFINED.

        Show
        seanpo03 Sean Po added a comment - - edited YARN-5384 .v10.patch replaces all occurrences of the default Priority.newInstance(0) with Priority.UNDEFINED.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v9.patch addresses all your comments Subru Krishnan. Thanks!

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v9.patch addresses all your comments Subru Krishnan . Thanks!
        Hide
        subru Subru Krishnan added a comment -

        Thanks Sean Po for addressing my comments.

        The latest patch is very close, have a few minor suggestions:

        • In YARN Priority is a inverse value, i.e. lower the value higher the absolute Priority. Can you kindly update the docs accordingly.
        • This statement is not fully correct:

          Note that a recurring reservation will implicitly have the highest possible priority

          Recurring reservations are always higher priority than non-recurring ones and we compare Priority within each group as described above.

        • We should use Priority.UNDEFINED if Priority is not specified.
        • All the API changes related to Priority should be marked unstable. Can you also update the ones related to recurrence and name as they are currently incorrectly annotated.
        Show
        subru Subru Krishnan added a comment - Thanks Sean Po for addressing my comments. The latest patch is very close, have a few minor suggestions: In YARN Priority is a inverse value, i.e. lower the value higher the absolute Priority. Can you kindly update the docs accordingly. This statement is not fully correct: Note that a recurring reservation will implicitly have the highest possible priority Recurring reservations are always higher priority than non-recurring ones and we compare Priority within each group as described above. We should use Priority.UNDEFINED if Priority is not specified. All the API changes related to Priority should be marked unstable . Can you also update the ones related to recurrence and name as they are currently incorrectly annotated.
        Hide
        hadoopqa Hadoop QA added a comment -
        +1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 14s 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 1m 4s Maven dependency ordering for branch
        +1 mvninstall 7m 51s trunk passed
        +1 compile 2m 34s trunk passed
        +1 checkstyle 0m 48s trunk passed
        +1 mvnsite 2m 3s trunk passed
        +1 mvneclipse 0m 52s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 23s trunk passed
        +1 javadoc 1m 22s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 1m 43s the patch passed
        +1 compile 2m 49s the patch passed
        +1 cc 2m 49s the patch passed
        +1 javac 2m 49s the patch passed
        +1 checkstyle 0m 43s the patch passed
        +1 mvnsite 1m 57s the patch passed
        +1 mvneclipse 0m 44s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 15s the patch passed
        +1 javadoc 1m 6s the patch passed
        +1 unit 0m 23s hadoop-yarn-api in the patch passed.
        +1 unit 2m 16s hadoop-yarn-common in the patch passed.
        +1 unit 35m 7s hadoop-yarn-server-resourcemanager in the patch passed.
        +1 unit 0m 9s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 18s The patch does not generate ASF License warnings.
        71m 47s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831001/YARN-5384.v8.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 702e84931121 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 10be459
        Default Java 1.8.0_101
        findbugs v3.0.0
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13252/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13252/console
        Powered by Apache Yetus 0.3.0 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 14s 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 1m 4s Maven dependency ordering for branch +1 mvninstall 7m 51s trunk passed +1 compile 2m 34s trunk passed +1 checkstyle 0m 48s trunk passed +1 mvnsite 2m 3s trunk passed +1 mvneclipse 0m 52s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 23s trunk passed +1 javadoc 1m 22s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 1m 43s the patch passed +1 compile 2m 49s the patch passed +1 cc 2m 49s the patch passed +1 javac 2m 49s the patch passed +1 checkstyle 0m 43s the patch passed +1 mvnsite 1m 57s the patch passed +1 mvneclipse 0m 44s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 15s the patch passed +1 javadoc 1m 6s the patch passed +1 unit 0m 23s hadoop-yarn-api in the patch passed. +1 unit 2m 16s hadoop-yarn-common in the patch passed. +1 unit 35m 7s hadoop-yarn-server-resourcemanager in the patch passed. +1 unit 0m 9s hadoop-yarn-site in the patch passed. +1 asflicense 0m 18s The patch does not generate ASF License warnings. 71m 47s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12831001/YARN-5384.v8.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 702e84931121 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 10be459 Default Java 1.8.0_101 findbugs v3.0.0 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13252/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13252/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v8.patch removes checkstyle issues introduced by YARN-5384.v7.patch.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v8.patch removes checkstyle issues introduced by YARN-5384 .v7.patch.
        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 11s Maven dependency ordering for branch
        +1 mvninstall 8m 10s trunk passed
        +1 compile 2m 45s trunk passed
        +1 checkstyle 0m 44s trunk passed
        +1 mvnsite 2m 6s trunk passed
        +1 mvneclipse 0m 55s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 24s trunk passed
        +1 javadoc 1m 23s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 1m 45s the patch passed
        +1 compile 2m 42s the patch passed
        +1 cc 2m 42s the patch passed
        +1 javac 2m 42s the patch passed
        -1 checkstyle 0m 41s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31)
        +1 mvnsite 2m 1s the patch passed
        +1 mvneclipse 0m 50s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 52s the patch passed
        +1 javadoc 1m 13s the patch passed
        +1 unit 0m 23s hadoop-yarn-api in the patch passed.
        +1 unit 2m 25s hadoop-yarn-common in the patch passed.
        +1 unit 35m 13s hadoop-yarn-server-resourcemanager in the patch passed.
        +1 unit 0m 7s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 19s The patch does not generate ASF License warnings.
        72m 34s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830946/YARN-5384.v7.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 569348c0624d 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 236ac77
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13250/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13250/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13250/console
        Powered by Apache Yetus 0.3.0 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 11s Maven dependency ordering for branch +1 mvninstall 8m 10s trunk passed +1 compile 2m 45s trunk passed +1 checkstyle 0m 44s trunk passed +1 mvnsite 2m 6s trunk passed +1 mvneclipse 0m 55s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 24s trunk passed +1 javadoc 1m 23s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 1m 45s the patch passed +1 compile 2m 42s the patch passed +1 cc 2m 42s the patch passed +1 javac 2m 42s the patch passed -1 checkstyle 0m 41s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31) +1 mvnsite 2m 1s the patch passed +1 mvneclipse 0m 50s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 52s the patch passed +1 javadoc 1m 13s the patch passed +1 unit 0m 23s hadoop-yarn-api in the patch passed. +1 unit 2m 25s hadoop-yarn-common in the patch passed. +1 unit 35m 13s hadoop-yarn-server-resourcemanager in the patch passed. +1 unit 0m 7s hadoop-yarn-site in the patch passed. +1 asflicense 0m 19s The patch does not generate ASF License warnings. 72m 34s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830946/YARN-5384.v7.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 569348c0624d 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 236ac77 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13250/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13250/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13250/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v7.patch fixes the test failures in TestGreedyReservationAgent.java.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v7.patch fixes the test failures in TestGreedyReservationAgent.java.
        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 1m 9s Maven dependency ordering for branch
        +1 mvninstall 7m 4s trunk passed
        +1 compile 2m 18s trunk passed
        +1 checkstyle 0m 40s trunk passed
        +1 mvnsite 1m 59s trunk passed
        +1 mvneclipse 0m 50s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 2m 53s trunk passed
        +1 javadoc 1m 12s trunk passed
        0 mvndep 0m 8s Maven dependency ordering for patch
        +1 mvninstall 1m 24s the patch passed
        +1 compile 2m 27s the patch passed
        +1 cc 2m 27s the patch passed
        +1 javac 2m 27s the patch passed
        +1 checkstyle 0m 38s the patch passed
        +1 mvnsite 1m 41s the patch passed
        +1 mvneclipse 0m 42s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 12s the patch passed
        +1 javadoc 1m 7s the patch passed
        +1 unit 0m 24s hadoop-yarn-api in the patch passed.
        +1 unit 2m 18s hadoop-yarn-common in the patch passed.
        -1 unit 33m 50s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 unit 0m 6s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 20s The patch does not generate ASF License warnings.
        67m 34s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830828/YARN-5384.v6.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux f8a163fd0cc5 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 47f8092
        Default Java 1.8.0_101
        findbugs v3.0.0
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13246/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13246/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13246/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13246/console
        Powered by Apache Yetus 0.3.0 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 1m 9s Maven dependency ordering for branch +1 mvninstall 7m 4s trunk passed +1 compile 2m 18s trunk passed +1 checkstyle 0m 40s trunk passed +1 mvnsite 1m 59s trunk passed +1 mvneclipse 0m 50s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 2m 53s trunk passed +1 javadoc 1m 12s trunk passed 0 mvndep 0m 8s Maven dependency ordering for patch +1 mvninstall 1m 24s the patch passed +1 compile 2m 27s the patch passed +1 cc 2m 27s the patch passed +1 javac 2m 27s the patch passed +1 checkstyle 0m 38s the patch passed +1 mvnsite 1m 41s the patch passed +1 mvneclipse 0m 42s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 12s the patch passed +1 javadoc 1m 7s the patch passed +1 unit 0m 24s hadoop-yarn-api in the patch passed. +1 unit 2m 18s hadoop-yarn-common in the patch passed. -1 unit 33m 50s hadoop-yarn-server-resourcemanager in the patch failed. +1 unit 0m 6s hadoop-yarn-site in the patch passed. +1 asflicense 0m 20s The patch does not generate ASF License warnings. 67m 34s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830828/YARN-5384.v6.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux f8a163fd0cc5 3.13.0-93-generic #140-Ubuntu SMP Mon Jul 18 21:21:05 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 47f8092 Default Java 1.8.0_101 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/13246/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13246/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13246/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13246/console Powered by Apache Yetus 0.3.0 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 14s 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 1m 1s Maven dependency ordering for branch
        +1 mvninstall 9m 19s trunk passed
        +1 compile 3m 14s trunk passed
        +1 checkstyle 0m 47s trunk passed
        +1 mvnsite 2m 13s trunk passed
        +1 mvneclipse 1m 7s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 4m 2s trunk passed
        +1 javadoc 1m 34s trunk passed
        0 mvndep 0m 10s Maven dependency ordering for patch
        +1 mvninstall 1m 50s the patch passed
        +1 compile 2m 54s the patch passed
        +1 cc 2m 54s the patch passed
        +1 javac 2m 54s the patch passed
        -1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31)
        +1 mvnsite 1m 54s the patch passed
        +1 mvneclipse 0m 48s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 4m 8s the patch passed
        +1 javadoc 1m 27s the patch passed
        +1 unit 0m 32s hadoop-yarn-api in the patch passed.
        +1 unit 2m 43s hadoop-yarn-common in the patch passed.
        -1 unit 39m 30s hadoop-yarn-server-resourcemanager in the patch failed.
        +1 unit 0m 8s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 19s The patch does not generate ASF License warnings.
        81m 45s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer
          hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830809/YARN-5384.v5.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 1ef35a0693f8 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 / 47f8092
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13244/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13244/console
        Powered by Apache Yetus 0.3.0 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 14s 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 1m 1s Maven dependency ordering for branch +1 mvninstall 9m 19s trunk passed +1 compile 3m 14s trunk passed +1 checkstyle 0m 47s trunk passed +1 mvnsite 2m 13s trunk passed +1 mvneclipse 1m 7s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 4m 2s trunk passed +1 javadoc 1m 34s trunk passed 0 mvndep 0m 10s Maven dependency ordering for patch +1 mvninstall 1m 50s the patch passed +1 compile 2m 54s the patch passed +1 cc 2m 54s the patch passed +1 javac 2m 54s the patch passed -1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 31 unchanged - 0 fixed = 32 total (was 31) +1 mvnsite 1m 54s the patch passed +1 mvneclipse 0m 48s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 4m 8s the patch passed +1 javadoc 1m 27s the patch passed +1 unit 0m 32s hadoop-yarn-api in the patch passed. +1 unit 2m 43s hadoop-yarn-common in the patch passed. -1 unit 39m 30s hadoop-yarn-server-resourcemanager in the patch failed. +1 unit 0m 8s hadoop-yarn-site in the patch passed. +1 asflicense 0m 19s The patch does not generate ASF License warnings. 81m 45s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer   hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830809/YARN-5384.v5.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 1ef35a0693f8 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 / 47f8092 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13244/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13244/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13244/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v6.patch fixes the checkstyle issues.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v6.patch fixes the checkstyle issues.
        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 2 new or modified test files.
        0 mvndep 0m 14s Maven dependency ordering for branch
        +1 mvninstall 7m 45s trunk passed
        +1 compile 2m 22s trunk passed
        +1 checkstyle 0m 41s trunk passed
        +1 mvnsite 2m 15s trunk passed
        +1 mvneclipse 1m 5s trunk passed
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 3m 20s trunk passed
        +1 javadoc 1m 28s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 1m 42s the patch passed
        +1 compile 2m 15s the patch passed
        +1 cc 2m 15s the patch passed
        +1 javac 2m 15s the patch passed
        -1 checkstyle 0m 37s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 132 unchanged - 0 fixed = 133 total (was 132)
        +1 mvnsite 2m 7s the patch passed
        +1 mvneclipse 0m 58s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site
        +1 findbugs 4m 3s the patch passed
        +1 javadoc 1m 39s the patch passed
        +1 unit 0m 30s hadoop-yarn-api in the patch passed.
        +1 unit 2m 29s hadoop-yarn-common in the patch passed.
        -1 unit 35m 1s hadoop-yarn-server-resourcemanager in the patch failed.
        -1 unit 17m 0s hadoop-yarn-client in the patch failed.
        +1 unit 0m 7s hadoop-yarn-site in the patch passed.
        +1 asflicense 0m 19s The patch does not generate ASF License warnings.
        89m 28s



        Reason Tests
        Failed junit tests hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent
          hadoop.yarn.server.resourcemanager.scheduler.fair.TestContinuousScheduling
          hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer
          hadoop.yarn.client.api.impl.TestYarnClient



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830803/YARN-5384.v4.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 7d1700e1a28a 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
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 84c6264
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13242/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13242/console
        Powered by Apache Yetus 0.3.0 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 2 new or modified test files. 0 mvndep 0m 14s Maven dependency ordering for branch +1 mvninstall 7m 45s trunk passed +1 compile 2m 22s trunk passed +1 checkstyle 0m 41s trunk passed +1 mvnsite 2m 15s trunk passed +1 mvneclipse 1m 5s trunk passed 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 3m 20s trunk passed +1 javadoc 1m 28s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 1m 42s the patch passed +1 compile 2m 15s the patch passed +1 cc 2m 15s the patch passed +1 javac 2m 15s the patch passed -1 checkstyle 0m 37s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 132 unchanged - 0 fixed = 133 total (was 132) +1 mvnsite 2m 7s the patch passed +1 mvneclipse 0m 58s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. 0 findbugs 0m 0s Skipped patched modules with no Java source: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site +1 findbugs 4m 3s the patch passed +1 javadoc 1m 39s the patch passed +1 unit 0m 30s hadoop-yarn-api in the patch passed. +1 unit 2m 29s hadoop-yarn-common in the patch passed. -1 unit 35m 1s hadoop-yarn-server-resourcemanager in the patch failed. -1 unit 17m 0s hadoop-yarn-client in the patch failed. +1 unit 0m 7s hadoop-yarn-site in the patch passed. +1 asflicense 0m 19s The patch does not generate ASF License warnings. 89m 28s Reason Tests Failed junit tests hadoop.yarn.server.resourcemanager.reservation.planning.TestGreedyReservationAgent   hadoop.yarn.server.resourcemanager.scheduler.fair.TestContinuousScheduling   hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer   hadoop.yarn.client.api.impl.TestYarnClient Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830803/YARN-5384.v4.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 7d1700e1a28a 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 Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 84c6264 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt https://builds.apache.org/job/PreCommit-YARN-Build/13242/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13242/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13242/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        Thanks Subru Krishnan for your comments! YARN-5384.v5.patch addresses all your comments, and removes some unrelated changes from YARN-5384.v4.patch,

        Show
        seanpo03 Sean Po added a comment - Thanks Subru Krishnan for your comments! YARN-5384 .v5.patch addresses all your comments, and removes some unrelated changes from YARN-5384 .v4.patch,
        Hide
        hadoopqa Hadoop QA added a comment -
        +1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 15s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 6 new or modified test files.
        0 mvndep 0m 9s Maven dependency ordering for branch
        +1 mvninstall 7m 25s trunk passed
        +1 compile 2m 18s trunk passed
        +1 checkstyle 0m 41s trunk passed
        +1 mvnsite 1m 58s trunk passed
        +1 mvneclipse 0m 56s trunk passed
        +1 findbugs 3m 25s trunk passed
        +1 javadoc 1m 29s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 1m 42s the patch passed
        +1 compile 2m 26s the patch passed
        +1 cc 2m 26s the patch passed
        +1 javac 2m 26s the patch passed
        +1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 0 new + 220 unchanged - 1 fixed = 220 total (was 221)
        +1 mvnsite 2m 24s the patch passed
        +1 mvneclipse 0m 57s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 4m 47s the patch passed
        +1 javadoc 1m 24s the patch passed
        +1 unit 0m 23s hadoop-yarn-api in the patch passed.
        +1 unit 2m 46s hadoop-yarn-common in the patch passed.
        +1 unit 35m 26s hadoop-yarn-server-resourcemanager in the patch passed.
        +1 unit 16m 10s hadoop-yarn-client in the patch passed.
        +1 asflicense 0m 17s The patch does not generate ASF License warnings.
        89m 7s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830583/YARN-5384.v3.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 576986e1c3f9 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
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 2acfb1e
        Default Java 1.8.0_101
        findbugs v3.0.0
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13232/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13232/console
        Powered by Apache Yetus 0.3.0 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 15s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 6 new or modified test files. 0 mvndep 0m 9s Maven dependency ordering for branch +1 mvninstall 7m 25s trunk passed +1 compile 2m 18s trunk passed +1 checkstyle 0m 41s trunk passed +1 mvnsite 1m 58s trunk passed +1 mvneclipse 0m 56s trunk passed +1 findbugs 3m 25s trunk passed +1 javadoc 1m 29s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 1m 42s the patch passed +1 compile 2m 26s the patch passed +1 cc 2m 26s the patch passed +1 javac 2m 26s the patch passed +1 checkstyle 0m 40s hadoop-yarn-project/hadoop-yarn: The patch generated 0 new + 220 unchanged - 1 fixed = 220 total (was 221) +1 mvnsite 2m 24s the patch passed +1 mvneclipse 0m 57s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 4m 47s the patch passed +1 javadoc 1m 24s the patch passed +1 unit 0m 23s hadoop-yarn-api in the patch passed. +1 unit 2m 46s hadoop-yarn-common in the patch passed. +1 unit 35m 26s hadoop-yarn-server-resourcemanager in the patch passed. +1 unit 16m 10s hadoop-yarn-client in the patch passed. +1 asflicense 0m 17s The patch does not generate ASF License warnings. 89m 7s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830583/YARN-5384.v3.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 576986e1c3f9 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 Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 2acfb1e Default Java 1.8.0_101 findbugs v3.0.0 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13232/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13232/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        subru Subru Krishnan added a comment -

        Thanks Sean Po for the patch.

        I looked at it & have a few comments:

        • Please use PriorityProto instead of int directly to be aligned with rest of YARN APIs.
        • Can you elaborate on Priority in the API Javadocs, i.e. what defines a higher priority and how do we deal with it and context with recurrence.
        • It should be cleaner to add a new ReservationDefinition::newInstance that takes in Priority to avoid changes in all the test classes.
        • What about corresponding documentation for Priority ? I am fine if you want to create a separate JIRA for it.
        • I think it makes sense to add priority to ReservationDefinitionPBImpl::toString.
        Show
        subru Subru Krishnan added a comment - Thanks Sean Po for the patch. I looked at it & have a few comments: Please use PriorityProto instead of int directly to be aligned with rest of YARN APIs. Can you elaborate on Priority in the API Javadocs, i.e. what defines a higher priority and how do we deal with it and context with recurrence . It should be cleaner to add a new ReservationDefinition::newInstance that takes in Priority to avoid changes in all the test classes. What about corresponding documentation for Priority ? I am fine if you want to create a separate JIRA for it. I think it makes sense to add priority to ReservationDefinitionPBImpl::toString .
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v3.patch fixes the checkstyle issue with an unused import. However, ReservationDefinition.java does not have whitespace at line 184, nor was it introduced with YARN-5384.v2.patch of the patch so the issue is ignored for now.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v3.patch fixes the checkstyle issue with an unused import. However, ReservationDefinition.java does not have whitespace at line 184, nor was it introduced with YARN-5384 .v2.patch of the patch so the issue is ignored for now.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 18s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 6 new or modified test files.
        0 mvndep 0m 10s Maven dependency ordering for branch
        +1 mvninstall 7m 8s trunk passed
        +1 compile 2m 22s trunk passed
        +1 checkstyle 0m 40s trunk passed
        +1 mvnsite 2m 3s trunk passed
        +1 mvneclipse 0m 57s trunk passed
        +1 findbugs 3m 28s trunk passed
        +1 javadoc 1m 19s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 1m 39s the patch passed
        +1 compile 2m 23s the patch passed
        +1 cc 2m 23s the patch passed
        +1 javac 2m 23s the patch passed
        -1 checkstyle 0m 39s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 220 unchanged - 1 fixed = 221 total (was 221)
        +1 mvnsite 1m 52s the patch passed
        +1 mvneclipse 0m 49s the patch passed
        -1 whitespace 0m 0s The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix.
        +1 findbugs 3m 52s the patch passed
        +1 javadoc 1m 15s the patch passed
        +1 unit 0m 24s hadoop-yarn-api in the patch passed.
        +1 unit 2m 20s hadoop-yarn-common in the patch passed.
        +1 unit 33m 54s hadoop-yarn-server-resourcemanager in the patch passed.
        +1 unit 15m 59s hadoop-yarn-client in the patch passed.
        +1 asflicense 0m 17s The patch does not generate ASF License warnings.
        85m 0s



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830554/YARN-5384.v2.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux 6e8855fd86cf 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
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / 1831be8
        Default Java 1.8.0_101
        findbugs v3.0.0
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13228/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt
        whitespace https://builds.apache.org/job/PreCommit-YARN-Build/13228/artifact/patchprocess/whitespace-eol.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13228/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/13228/console
        Powered by Apache Yetus 0.3.0 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 18s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 6 new or modified test files. 0 mvndep 0m 10s Maven dependency ordering for branch +1 mvninstall 7m 8s trunk passed +1 compile 2m 22s trunk passed +1 checkstyle 0m 40s trunk passed +1 mvnsite 2m 3s trunk passed +1 mvneclipse 0m 57s trunk passed +1 findbugs 3m 28s trunk passed +1 javadoc 1m 19s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 1m 39s the patch passed +1 compile 2m 23s the patch passed +1 cc 2m 23s the patch passed +1 javac 2m 23s the patch passed -1 checkstyle 0m 39s hadoop-yarn-project/hadoop-yarn: The patch generated 1 new + 220 unchanged - 1 fixed = 221 total (was 221) +1 mvnsite 1m 52s the patch passed +1 mvneclipse 0m 49s the patch passed -1 whitespace 0m 0s The patch has 1 line(s) that end in whitespace. Use git apply --whitespace=fix. +1 findbugs 3m 52s the patch passed +1 javadoc 1m 15s the patch passed +1 unit 0m 24s hadoop-yarn-api in the patch passed. +1 unit 2m 20s hadoop-yarn-common in the patch passed. +1 unit 33m 54s hadoop-yarn-server-resourcemanager in the patch passed. +1 unit 15m 59s hadoop-yarn-client in the patch passed. +1 asflicense 0m 17s The patch does not generate ASF License warnings. 85m 0s Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12830554/YARN-5384.v2.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux 6e8855fd86cf 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 Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / 1831be8 Default Java 1.8.0_101 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/13228/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn.txt whitespace https://builds.apache.org/job/PreCommit-YARN-Build/13228/artifact/patchprocess/whitespace-eol.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/13228/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/13228/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v2.patch fixes the merge conflicts with latest trunk.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v2.patch fixes the merge conflicts with latest trunk.
        Hide
        hadoopqa Hadoop QA added a comment -
        -1 overall



        Vote Subsystem Runtime Comment
        0 reexec 0m 13s Docker mode activated.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 test4tests 0m 0s The patch appears to include 6 new or modified test files.
        0 mvndep 1m 0s Maven dependency ordering for branch
        +1 mvninstall 6m 54s trunk passed
        +1 compile 2m 22s trunk passed
        +1 checkstyle 0m 41s trunk passed
        +1 mvnsite 1m 58s trunk passed
        +1 mvneclipse 0m 55s trunk passed
        +1 findbugs 3m 21s trunk passed
        +1 javadoc 1m 18s trunk passed
        0 mvndep 0m 9s Maven dependency ordering for patch
        +1 mvninstall 1m 34s the patch passed
        +1 compile 2m 16s the patch passed
        +1 cc 2m 16s the patch passed
        +1 javac 2m 16s the patch passed
        +1 checkstyle 0m 38s the patch passed
        +1 mvnsite 1m 47s the patch passed
        +1 mvneclipse 0m 48s the patch passed
        +1 whitespace 0m 0s The patch has no whitespace issues.
        +1 findbugs 3m 42s the patch passed
        +1 javadoc 1m 9s the patch passed
        +1 unit 0m 23s hadoop-yarn-api in the patch passed.
        -1 unit 2m 13s hadoop-yarn-common in the patch failed.
        +1 unit 33m 6s hadoop-yarn-server-resourcemanager in the patch passed.
        -1 unit 8m 49s hadoop-yarn-client in the patch failed.
        +1 asflicense 0m 19s The patch does not generate ASF License warnings.
        76m 39s



        Reason Tests
        Failed junit tests hadoop.yarn.logaggregation.TestAggregatedLogFormat
          hadoop.yarn.client.api.impl.TestYarnClient



        Subsystem Report/Notes
        Docker Image:yetus/hadoop:9560f25
        JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12821940/YARN-5384.v1.patch
        JIRA Issue YARN-5384
        Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc
        uname Linux bb5449a142d6 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
        Build tool maven
        Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh
        git revision trunk / a1f6564
        Default Java 1.8.0_101
        findbugs v3.0.0
        unit https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt
        unit https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
        unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/12636/testReport/
        modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/12636/console
        Powered by Apache Yetus 0.3.0 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 13s Docker mode activated. +1 @author 0m 0s The patch does not contain any @author tags. +1 test4tests 0m 0s The patch appears to include 6 new or modified test files. 0 mvndep 1m 0s Maven dependency ordering for branch +1 mvninstall 6m 54s trunk passed +1 compile 2m 22s trunk passed +1 checkstyle 0m 41s trunk passed +1 mvnsite 1m 58s trunk passed +1 mvneclipse 0m 55s trunk passed +1 findbugs 3m 21s trunk passed +1 javadoc 1m 18s trunk passed 0 mvndep 0m 9s Maven dependency ordering for patch +1 mvninstall 1m 34s the patch passed +1 compile 2m 16s the patch passed +1 cc 2m 16s the patch passed +1 javac 2m 16s the patch passed +1 checkstyle 0m 38s the patch passed +1 mvnsite 1m 47s the patch passed +1 mvneclipse 0m 48s the patch passed +1 whitespace 0m 0s The patch has no whitespace issues. +1 findbugs 3m 42s the patch passed +1 javadoc 1m 9s the patch passed +1 unit 0m 23s hadoop-yarn-api in the patch passed. -1 unit 2m 13s hadoop-yarn-common in the patch failed. +1 unit 33m 6s hadoop-yarn-server-resourcemanager in the patch passed. -1 unit 8m 49s hadoop-yarn-client in the patch failed. +1 asflicense 0m 19s The patch does not generate ASF License warnings. 76m 39s Reason Tests Failed junit tests hadoop.yarn.logaggregation.TestAggregatedLogFormat   hadoop.yarn.client.api.impl.TestYarnClient Subsystem Report/Notes Docker Image:yetus/hadoop:9560f25 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12821940/YARN-5384.v1.patch JIRA Issue YARN-5384 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle cc uname Linux bb5449a142d6 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 Build tool maven Personality /testptch/hadoop/patchprocess/precommit/personality/provided.sh git revision trunk / a1f6564 Default Java 1.8.0_101 findbugs v3.0.0 unit https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-common.txt https://builds.apache.org/job/PreCommit-YARN-Build/12636/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/12636/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-api hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn Console output https://builds.apache.org/job/PreCommit-YARN-Build/12636/console Powered by Apache Yetus 0.3.0 http://yetus.apache.org This message was automatically generated.
        Hide
        seanpo03 Sean Po added a comment -

        YARN-5384.v1.patch adds priority to the ReservationSystem submission API by adding another field to the ReservationDefinition class.

        Show
        seanpo03 Sean Po added a comment - YARN-5384 .v1.patch adds priority to the ReservationSystem submission API by adding another field to the ReservationDefinition class.

          People

          • Assignee:
            seanpo03 Sean Po
            Reporter:
            seanpo03 Sean Po
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development