Uploaded image for project: 'Hadoop YARN'
  1. Hadoop YARN
  2. YARN-4628

Display application priority in yarn top

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • None
    • 2.9.0, 3.0.0-alpha1
    • None
    • None

    Description

      In Topcli support Apppriority

      Attachments

        1. 0003-YARN-4628.patch
          3 kB
          Bibin Chundatt
        2. 0002-YARN-4628.patch
          2 kB
          Bibin Chundatt
        3. 0001-YARN-4628.patch
          3 kB
          Bibin Chundatt

        Activity

          bibinchundatt Bibin Chundatt added a comment -

          sunilg/rohithsharma
          Attaching patch for review

          bibinchundatt Bibin Chundatt added a comment - sunilg / rohithsharma Attaching patch for review
          vvasudev Varun Vasudev added a comment -

          Thanks for the patch bibinchundatt. Some minor fixes required -

          1)

          -import com.google.common.cache.Cache;
          -import com.google.common.cache.CacheBuilder;
          
          +import com.google.common.cache.Cache;
          +import com.google.common.cache.CacheBuilder;
          +
          

          This change is not required.

          2)
          Rename priorityComparator to AppPriorityComparator

          Also, it looks like the patch hasn't gone through Jenkins - can you please submit the patch for Jenkins once you make the changes?

          vvasudev Varun Vasudev added a comment - Thanks for the patch bibinchundatt . Some minor fixes required - 1) - import com.google.common.cache.Cache; - import com.google.common.cache.CacheBuilder; + import com.google.common.cache.Cache; + import com.google.common.cache.CacheBuilder; + This change is not required. 2) Rename priorityComparator to AppPriorityComparator Also, it looks like the patch hasn't gone through Jenkins - can you please submit the patch for Jenkins once you make the changes?
          bibinchundatt Bibin Chundatt added a comment -

          vvasudev

          Thank you for review comments. Attaching patch after handling comments

          bibinchundatt Bibin Chundatt added a comment - vvasudev Thank you for review comments. Attaching patch after handling comments
          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 doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 mvninstall 6m 45s trunk passed
          +1 compile 0m 16s trunk passed with JDK v1.8.0_72
          +1 compile 0m 17s trunk passed with JDK v1.7.0_95
          +1 checkstyle 0m 16s trunk passed
          +1 mvnsite 0m 22s trunk passed
          +1 mvneclipse 0m 13s trunk passed
          +1 findbugs 0m 34s trunk passed
          +1 javadoc 0m 18s trunk passed with JDK v1.8.0_72
          +1 javadoc 0m 17s trunk passed with JDK v1.7.0_95
          +1 mvninstall 0m 20s the patch passed
          +1 compile 0m 19s the patch passed with JDK v1.8.0_72
          +1 javac 0m 19s the patch passed
          +1 compile 0m 17s the patch passed with JDK v1.7.0_95
          +1 javac 0m 17s the patch passed
          -1 checkstyle 0m 12s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: patch generated 1 new + 151 unchanged - 0 fixed = 152 total (was 151)
          +1 mvnsite 0m 20s the patch passed
          +1 mvneclipse 0m 10s the patch passed
          +1 whitespace 0m 0s Patch has no whitespace issues.
          +1 findbugs 0m 45s the patch passed
          +1 javadoc 0m 13s the patch passed with JDK v1.8.0_72
          +1 javadoc 0m 14s the patch passed with JDK v1.7.0_95
          -1 unit 64m 29s hadoop-yarn-client in the patch failed with JDK v1.8.0_72.
          -1 unit 64m 40s hadoop-yarn-client in the patch failed with JDK v1.7.0_95.
          +1 asflicense 0m 18s Patch does not generate ASF License warnings.
          142m 45s



          Reason Tests
          JDK v1.8.0_72 Failed junit tests hadoop.yarn.client.TestGetGroups
          JDK v1.8.0_72 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI
            org.apache.hadoop.yarn.client.api.impl.TestYarnClient
            org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
            org.apache.hadoop.yarn.client.api.impl.TestNMClient
          JDK v1.7.0_95 Failed junit tests hadoop.yarn.client.TestGetGroups
          JDK v1.7.0_95 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI
            org.apache.hadoop.yarn.client.api.impl.TestYarnClient
            org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
            org.apache.hadoop.yarn.client.api.impl.TestNMClient



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:0ca8df7
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12787100/0002-YARN-4628.patch
          JIRA Issue YARN-4628
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 25fcee9cf96e 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 / 60d2011
          Default Java 1.7.0_95
          Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_72 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
          unit https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_72.txt
          unit https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_95.txt
          unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_72.txt https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_95.txt
          JDK v1.7.0_95 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/10535/testReport/
          modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client
          Max memory used 77MB
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/10535/console
          Powered by Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org

          This message was automatically generated.

          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 doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 mvninstall 6m 45s trunk passed +1 compile 0m 16s trunk passed with JDK v1.8.0_72 +1 compile 0m 17s trunk passed with JDK v1.7.0_95 +1 checkstyle 0m 16s trunk passed +1 mvnsite 0m 22s trunk passed +1 mvneclipse 0m 13s trunk passed +1 findbugs 0m 34s trunk passed +1 javadoc 0m 18s trunk passed with JDK v1.8.0_72 +1 javadoc 0m 17s trunk passed with JDK v1.7.0_95 +1 mvninstall 0m 20s the patch passed +1 compile 0m 19s the patch passed with JDK v1.8.0_72 +1 javac 0m 19s the patch passed +1 compile 0m 17s the patch passed with JDK v1.7.0_95 +1 javac 0m 17s the patch passed -1 checkstyle 0m 12s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: patch generated 1 new + 151 unchanged - 0 fixed = 152 total (was 151) +1 mvnsite 0m 20s the patch passed +1 mvneclipse 0m 10s the patch passed +1 whitespace 0m 0s Patch has no whitespace issues. +1 findbugs 0m 45s the patch passed +1 javadoc 0m 13s the patch passed with JDK v1.8.0_72 +1 javadoc 0m 14s the patch passed with JDK v1.7.0_95 -1 unit 64m 29s hadoop-yarn-client in the patch failed with JDK v1.8.0_72. -1 unit 64m 40s hadoop-yarn-client in the patch failed with JDK v1.7.0_95. +1 asflicense 0m 18s Patch does not generate ASF License warnings. 142m 45s Reason Tests JDK v1.8.0_72 Failed junit tests hadoop.yarn.client.TestGetGroups JDK v1.8.0_72 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI   org.apache.hadoop.yarn.client.api.impl.TestYarnClient   org.apache.hadoop.yarn.client.api.impl.TestAMRMClient   org.apache.hadoop.yarn.client.api.impl.TestNMClient JDK v1.7.0_95 Failed junit tests hadoop.yarn.client.TestGetGroups JDK v1.7.0_95 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI   org.apache.hadoop.yarn.client.api.impl.TestYarnClient   org.apache.hadoop.yarn.client.api.impl.TestAMRMClient   org.apache.hadoop.yarn.client.api.impl.TestNMClient Subsystem Report/Notes Docker Image:yetus/hadoop:0ca8df7 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12787100/0002-YARN-4628.patch JIRA Issue YARN-4628 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 25fcee9cf96e 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 / 60d2011 Default Java 1.7.0_95 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_72 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_72.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_95.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_72.txt https://builds.apache.org/job/PreCommit-YARN-Build/10535/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_95.txt JDK v1.7.0_95 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/10535/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client Max memory used 77MB Console output https://builds.apache.org/job/PreCommit-YARN-Build/10535/console Powered by Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          vvasudev Varun Vasudev added a comment -

          Thanks for the patch bibinchundatt. Couple of things -
          1)

           priority = appReport.getPriority().getPriority(); 

          appReport.getPriority() can return null - can you add a null check?

          2) Can you run the unit tests on your machine to confirm if they're passing?

          vvasudev Varun Vasudev added a comment - Thanks for the patch bibinchundatt . Couple of things - 1) priority = appReport.getPriority().getPriority(); appReport.getPriority() can return null - can you add a null check? 2) Can you run the unit tests on your machine to confirm if they're passing?
          bibinchundatt Bibin Chundatt added a comment -

          Test failures are not related to patch attached

          bibinchundatt Bibin Chundatt added a comment - Test failures are not related to patch attached
          bibinchundatt Bibin Chundatt added a comment -

          vvasudev
          Thank you for review . Will handle the same and upload patch soon

          bibinchundatt Bibin Chundatt added a comment - vvasudev Thank you for review . Will handle the same and upload patch soon
          sunilg Sunil G added a comment -

          Hi bibinchundatt
          Whether this cli will show "Higher" priority at the top. (because current comparator is in ascending order) ?

          sunilg Sunil G added a comment - Hi bibinchundatt Whether this cli will show "Higher" priority at the top. (because current comparator is in ascending order) ?
          vvasudev Varun Vasudev added a comment -

          Good point sunilg. Bibin, can you please change the comparator to address Sunil's comment?

          vvasudev Varun Vasudev added a comment - Good point sunilg . Bibin, can you please change the comparator to address Sunil's comment?
          bibinchundatt Bibin Chundatt added a comment -

          sunilg

          Thanks for review comment.

          By default if we are sorting on prority will be done in descending order since ascendingSort = false by default . (Higher priority first) .

              if (ascendingSort) {
                Collections.sort(appsInfo, comparator);
              } else {
                Collections.sort(appsInfo, Collections.reverseOrder(comparator));
              }
          

          Also we do have option for reversing in in top cli.

          vvasudev
          Handled latest review comments too. Attaching lated patch. Please do review

          bibinchundatt Bibin Chundatt added a comment - sunilg Thanks for review comment. By default if we are sorting on prority will be done in descending order since ascendingSort = false by default . (Higher priority first) . if (ascendingSort) { Collections.sort(appsInfo, comparator); } else { Collections.sort(appsInfo, Collections.reverseOrder(comparator)); } Also we do have option for reversing in in top cli. vvasudev Handled latest review comments too. Attaching lated patch. Please do review
          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 doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
          +1 mvninstall 7m 12s trunk passed
          +1 compile 0m 17s trunk passed with JDK v1.8.0_66
          +1 compile 0m 19s trunk passed with JDK v1.7.0_91
          +1 checkstyle 0m 16s trunk passed
          +1 mvnsite 0m 24s trunk passed
          +1 mvneclipse 0m 13s trunk passed
          +1 findbugs 0m 34s trunk passed
          +1 javadoc 0m 15s trunk passed with JDK v1.8.0_66
          +1 javadoc 0m 18s trunk passed with JDK v1.7.0_91
          +1 mvninstall 0m 19s the patch passed
          +1 compile 0m 14s the patch passed with JDK v1.8.0_66
          +1 javac 0m 14s the patch passed
          +1 compile 0m 17s the patch passed with JDK v1.7.0_91
          +1 javac 0m 17s the patch passed
          -1 checkstyle 0m 13s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: patch generated 1 new + 151 unchanged - 0 fixed = 152 total (was 151)
          +1 mvnsite 0m 23s the patch passed
          +1 mvneclipse 0m 11s the patch passed
          +1 whitespace 0m 0s Patch has no whitespace issues.
          +1 findbugs 0m 46s the patch passed
          +1 javadoc 0m 13s the patch passed with JDK v1.8.0_66
          +1 javadoc 0m 15s the patch passed with JDK v1.7.0_91
          -1 unit 64m 38s hadoop-yarn-client in the patch failed with JDK v1.8.0_66.
          -1 unit 64m 48s hadoop-yarn-client in the patch failed with JDK v1.7.0_91.
          +1 asflicense 0m 19s Patch does not generate ASF License warnings.
          143m 35s



          Reason Tests
          JDK v1.8.0_66 Failed junit tests hadoop.yarn.client.TestGetGroups
          JDK v1.8.0_66 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI
            org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
            org.apache.hadoop.yarn.client.api.impl.TestYarnClient
            org.apache.hadoop.yarn.client.api.impl.TestNMClient
          JDK v1.7.0_91 Failed junit tests hadoop.yarn.client.TestGetGroups
          JDK v1.7.0_91 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI
            org.apache.hadoop.yarn.client.api.impl.TestAMRMClient
            org.apache.hadoop.yarn.client.api.impl.TestYarnClient
            org.apache.hadoop.yarn.client.api.impl.TestNMClient



          Subsystem Report/Notes
          Docker Image:yetus/hadoop:0ca8df7
          JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12787296/0003-YARN-4628.patch
          JIRA Issue YARN-4628
          Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle
          uname Linux 4af8fa6f259b 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 / c3641ed
          Default Java 1.7.0_91
          Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_66 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_91
          findbugs v3.0.0
          checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt
          unit https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_66.txt
          unit https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_91.txt
          unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_66.txt https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_91.txt
          JDK v1.7.0_91 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/10546/testReport/
          modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client
          Max memory used 76MB
          Console output https://builds.apache.org/job/PreCommit-YARN-Build/10546/console
          Powered by Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org

          This message was automatically generated.

          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 doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 mvninstall 7m 12s trunk passed +1 compile 0m 17s trunk passed with JDK v1.8.0_66 +1 compile 0m 19s trunk passed with JDK v1.7.0_91 +1 checkstyle 0m 16s trunk passed +1 mvnsite 0m 24s trunk passed +1 mvneclipse 0m 13s trunk passed +1 findbugs 0m 34s trunk passed +1 javadoc 0m 15s trunk passed with JDK v1.8.0_66 +1 javadoc 0m 18s trunk passed with JDK v1.7.0_91 +1 mvninstall 0m 19s the patch passed +1 compile 0m 14s the patch passed with JDK v1.8.0_66 +1 javac 0m 14s the patch passed +1 compile 0m 17s the patch passed with JDK v1.7.0_91 +1 javac 0m 17s the patch passed -1 checkstyle 0m 13s hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client: patch generated 1 new + 151 unchanged - 0 fixed = 152 total (was 151) +1 mvnsite 0m 23s the patch passed +1 mvneclipse 0m 11s the patch passed +1 whitespace 0m 0s Patch has no whitespace issues. +1 findbugs 0m 46s the patch passed +1 javadoc 0m 13s the patch passed with JDK v1.8.0_66 +1 javadoc 0m 15s the patch passed with JDK v1.7.0_91 -1 unit 64m 38s hadoop-yarn-client in the patch failed with JDK v1.8.0_66. -1 unit 64m 48s hadoop-yarn-client in the patch failed with JDK v1.7.0_91. +1 asflicense 0m 19s Patch does not generate ASF License warnings. 143m 35s Reason Tests JDK v1.8.0_66 Failed junit tests hadoop.yarn.client.TestGetGroups JDK v1.8.0_66 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI   org.apache.hadoop.yarn.client.api.impl.TestAMRMClient   org.apache.hadoop.yarn.client.api.impl.TestYarnClient   org.apache.hadoop.yarn.client.api.impl.TestNMClient JDK v1.7.0_91 Failed junit tests hadoop.yarn.client.TestGetGroups JDK v1.7.0_91 Timed out junit tests org.apache.hadoop.yarn.client.cli.TestYarnCLI   org.apache.hadoop.yarn.client.api.impl.TestAMRMClient   org.apache.hadoop.yarn.client.api.impl.TestYarnClient   org.apache.hadoop.yarn.client.api.impl.TestNMClient Subsystem Report/Notes Docker Image:yetus/hadoop:0ca8df7 JIRA Patch URL https://issues.apache.org/jira/secure/attachment/12787296/0003-YARN-4628.patch JIRA Issue YARN-4628 Optional Tests asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle uname Linux 4af8fa6f259b 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 / c3641ed Default Java 1.7.0_91 Multi-JDK versions /usr/lib/jvm/java-8-oracle:1.8.0_66 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_91 findbugs v3.0.0 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/diff-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_66.txt unit https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_91.txt unit test logs https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.8.0_66.txt https://builds.apache.org/job/PreCommit-YARN-Build/10546/artifact/patchprocess/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-client-jdk1.7.0_91.txt JDK v1.7.0_91 Test Results https://builds.apache.org/job/PreCommit-YARN-Build/10546/testReport/ modules C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client Max memory used 76MB Console output https://builds.apache.org/job/PreCommit-YARN-Build/10546/console Powered by Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org This message was automatically generated.
          vvasudev Varun Vasudev added a comment - - edited

          +1. Committed to trunk and branch-2 with a minor modification - changed "#PRIOR" to "PRIOR".

          Thanks bibinchundatt!

          vvasudev Varun Vasudev added a comment - - edited +1. Committed to trunk and branch-2 with a minor modification - changed "#PRIOR" to "PRIOR". Thanks bibinchundatt !
          hudson Hudson added a comment -

          FAILURE: Integrated in Hadoop-trunk-Commit #9282 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9282/)
          YARN-4628. Display application priority in yarn top. Contributed by (vvasudev: rev 663a80031cf6f17151622fff327171dedd976df3)

          • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/TopCLI.java
          • hadoop-yarn-project/CHANGES.txt
          hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #9282 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9282/ ) YARN-4628 . Display application priority in yarn top. Contributed by (vvasudev: rev 663a80031cf6f17151622fff327171dedd976df3) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/TopCLI.java hadoop-yarn-project/CHANGES.txt
          bibinchundatt Bibin Chundatt added a comment -

          sunilg and vvasudev
          Thank you for review and commit.

          bibinchundatt Bibin Chundatt added a comment - sunilg and vvasudev Thank you for review and commit.

          People

            bibinchundatt Bibin Chundatt
            bibinchundatt Bibin Chundatt
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: