Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.7.1
    • Fix Version/s: 2.8.0, 3.0.0-alpha1
    • Component/s: webapp
    • Labels:
      None

      Description

      REST api support for application priority.

      • get/set priority of an application
      • get default priority of a queue
      • get cluster max priority
      1. YARN-3970.20150901-2.patch
        24 kB
        Naganarasimha G R
      2. YARN-3970.20150901-1.patch
        24 kB
        Naganarasimha G R
      3. YARN-3970.20150831-1.patch
        18 kB
        Naganarasimha G R
      4. YARN-3970.20150829-1.patch
        15 kB
        Naganarasimha G R
      5. YARN-3970.20150828-1.patch
        14 kB
        Naganarasimha G R

        Activity

        Hide
        rohithsharma Rohith Sharma K S added a comment -

        Offline Naganarasimha G R pinged for taking over this. Assigning to Naganarasimha G R. Expecting patch !!

        Show
        rohithsharma Rohith Sharma K S added a comment - Offline Naganarasimha G R pinged for taking over this. Assigning to Naganarasimha G R. Expecting patch !!
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks Rohith Sharma K S,
        As per the offline discussion we had, as CLI has not yet been handled for displaying "default priority of a queue" & "get cluster max priority", focusing only on set (more like update)Priority of an application through REST
        P.S. get is already handled as part of Applicationreport from getApps.
        REST URL which i am planning for updating priority of the application is : "/ws/v1/cluster/apps/{appid}/priority"
        And planning to invoke rm.getClientRMService().updateApplicationPriority() to achieve the App priority update.

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks Rohith Sharma K S , As per the offline discussion we had, as CLI has not yet been handled for displaying "default priority of a queue" & "get cluster max priority", focusing only on set (more like update)Priority of an application through REST P.S. get is already handled as part of Applicationreport from getApps. REST URL which i am planning for updating priority of the application is : "/ws/v1/cluster/apps/{appid}/priority" And planning to invoke rm.getClientRMService().updateApplicationPriority() to achieve the App priority update.
        Hide
        sunilg Sunil G added a comment -

        Hi Naga.
        Approach sounds good. Existing app reports will provide the running
        priority. And it's also good to verify whether app is in accepted state or
        running state before invoking scheduler api to change priority.

        On Fri, Aug 28, 2015, 11:43 AM Naganarasimha G R (JIRA) <jira@apache.org>

        Show
        sunilg Sunil G added a comment - Hi Naga. Approach sounds good. Existing app reports will provide the running priority. And it's also good to verify whether app is in accepted state or running state before invoking scheduler api to change priority. On Fri, Aug 28, 2015, 11:43 AM Naganarasimha G R (JIRA) <jira@apache.org>
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks for replying Sunil G,

        And it's also good to verify whether app is in accepted state or running state before invoking scheduler api to change priority

        On calling rm.getClientRMService().updateApplicationPriority() the above check will be taken care inside it. Also all acl related checks also will be handled.

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks for replying Sunil G , And it's also good to verify whether app is in accepted state or running state before invoking scheduler api to change priority On calling rm.getClientRMService().updateApplicationPriority() the above check will be taken care inside it. Also all acl related checks also will be handled.
        Hide
        sunilg Sunil G added a comment -

        Hi Naga.
        Yes. I could see that you are planning to use getClientRMService.
        That saves the direct api invocation to AbstractYarnScheduler. This looks
        fine as all validation are handled.
        Thank You.

        On Fri, Aug 28, 2015, 12:38 PM Naganarasimha G R (JIRA) <jira@apache.org>

        Show
        sunilg Sunil G added a comment - Hi Naga. Yes. I could see that you are planning to use getClientRMService . That saves the direct api invocation to AbstractYarnScheduler. This looks fine as all validation are handled. Thank You. On Fri, Aug 28, 2015, 12:38 PM Naganarasimha G R (JIRA) <jira@apache.org>
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Hi Sunil G & Rohith Sharma K S, Attaching the first patch as per the prev discussion and also there was one issue in CapacityScheduler.updateApplicationPriority, suppose already application is running with the Cluster Max Priority and user specifies some priority greater than MaxPriority, unnecessarily RMStatestore update and queue's treeset is updated with MaxPriority again.

        Show
        Naganarasimha Naganarasimha G R added a comment - Hi Sunil G & Rohith Sharma K S , Attaching the first patch as per the prev discussion and also there was one issue in CapacityScheduler.updateApplicationPriority, suppose already application is running with the Cluster Max Priority and user specifies some priority greater than MaxPriority, unnecessarily RMStatestore update and queue's treeset is updated with MaxPriority again.
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 16m 52s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 8m 8s There were no new javac warning messages.
        +1 javadoc 10m 6s There were no new javadoc warning messages.
        -1 release audit 0m 20s The applied patch generated 1 release audit warnings.
        -1 checkstyle 0m 52s The applied patch generated 5 new checkstyle issues (total was 164, now 169).
        +1 whitespace 0m 2s The patch has no lines that end in whitespace.
        +1 install 1m 31s mvn install still works.
        +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
        +1 findbugs 1m 31s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 54m 18s Tests passed in hadoop-yarn-server-resourcemanager.
            94m 18s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12753002/YARN-3970.20150828-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / beb65c9
        Release Audit https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/patchReleaseAuditProblems.txt
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt
        hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8934/testReport/
        Java 1.7.0_55
        uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8934/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 16m 52s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 8m 8s There were no new javac warning messages. +1 javadoc 10m 6s There were no new javadoc warning messages. -1 release audit 0m 20s The applied patch generated 1 release audit warnings. -1 checkstyle 0m 52s The applied patch generated 5 new checkstyle issues (total was 164, now 169). +1 whitespace 0m 2s The patch has no lines that end in whitespace. +1 install 1m 31s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 1m 31s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 54m 18s Tests passed in hadoop-yarn-server-resourcemanager.     94m 18s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12753002/YARN-3970.20150828-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / beb65c9 Release Audit https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/patchReleaseAuditProblems.txt checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8934/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8934/testReport/ Java 1.7.0_55 uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/8934/console This message was automatically generated.
        Hide
        sunilg Sunil G added a comment -

        Thank you Naganarasimha G R

        Yes. We can have the improvement to save an extra save to statestore. And the fix for the same looks good.

        Few comments

        1. updateAppPriority --> updateApplicationPriority. I prefer we can have full expanded name here as its a separate class to identify a web app object.
        2. priority.getPriority() != targetPriority.getPriority() We could use !priority.equals(targetPriority)
        3.

        +    AppPriority effectivePriority = new AppPriority(
        +        app.getApplicationSubmissionContext().getPriority().getPriority());
        

        If app.getApplicationSubmissionContext().getPriority() is NULL, we will get n NPE here.

        Show
        sunilg Sunil G added a comment - Thank you Naganarasimha G R Yes. We can have the improvement to save an extra save to statestore. And the fix for the same looks good. Few comments 1. updateAppPriority --> updateApplicationPriority. I prefer we can have full expanded name here as its a separate class to identify a web app object. 2. priority.getPriority() != targetPriority.getPriority() We could use !priority.equals(targetPriority) 3. + AppPriority effectivePriority = new AppPriority( + app.getApplicationSubmissionContext().getPriority().getPriority()); If app.getApplicationSubmissionContext().getPriority() is NULL, we will get n NPE here.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks Sunil G for the review comments,

        priority.getPriority() != targetPriority.getPriority() We could use !priority.equals(targetPriority)

        targetPriority is of type "org.apache.hadoop.yarn.server.resourcemanager.webapp.dao.AppPriority" and priority is of type "org.apache.hadoop.yarn.api.records.Priority". so it cannot be compared as per your suggestion.

        If app.getApplicationSubmissionContext().getPriority() is NULL, we will get n NPE here.

        Well went through the flow again, i think i have got the if clause wrong here, what i am trying to check here is, if target priority is same as the current priority then just return as success, so i can just return back with the target priority. Also i think i need to additionally validate whether target priority is not null. So after correction of all this, NPE is not possible here but other checks i will be adding as follows

            if (targetPriority == null) {
              throw new YarnException("Target Priority cannot be null");
            }
            . . . 
            Priority priority = app.getApplicationSubmissionContext().getPriority();
            if (priority == null
                || priority.getPriority() != targetPriority.getPriority()) {
              return modifyApplicationPriority(app, callerUGI,
                  targetPriority.getPriority());
            }
            return Response.status(Status.OK).entity(targetPriority).build();
        
        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks Sunil G for the review comments, priority.getPriority() != targetPriority.getPriority() We could use !priority.equals(targetPriority) targetPriority is of type "org.apache.hadoop.yarn.server.resourcemanager.webapp.dao.AppPriority" and priority is of type "org.apache.hadoop.yarn.api.records.Priority". so it cannot be compared as per your suggestion. If app.getApplicationSubmissionContext().getPriority() is NULL, we will get n NPE here. Well went through the flow again, i think i have got the if clause wrong here, what i am trying to check here is, if target priority is same as the current priority then just return as success, so i can just return back with the target priority. Also i think i need to additionally validate whether target priority is not null. So after correction of all this, NPE is not possible here but other checks i will be adding as follows if (targetPriority == null ) { throw new YarnException( "Target Priority cannot be null " ); } . . . Priority priority = app.getApplicationSubmissionContext().getPriority(); if (priority == null || priority.getPriority() != targetPriority.getPriority()) { return modifyApplicationPriority(app, callerUGI, targetPriority.getPriority()); } return Response.status(Status.OK).entity(targetPriority).build();
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        -1 pre-patch 15m 28s Findbugs (version ) appears to be broken on trunk.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 7m 42s There were no new javac warning messages.
        +1 javadoc 9m 49s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        +1 checkstyle 0m 29s There were no new checkstyle issues.
        +1 whitespace 0m 2s The patch has no lines that end in whitespace.
        +1 install 1m 31s mvn install still works.
        +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse.
        +1 findbugs 1m 30s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 54m 46s Tests passed in hadoop-yarn-server-resourcemanager.
            92m 17s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12753127/YARN-3970.20150829-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / e2c9b28
        hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8942/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8942/testReport/
        Java 1.7.0_55
        uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8942/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment -1 pre-patch 15m 28s Findbugs (version ) appears to be broken on trunk. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 42s There were no new javac warning messages. +1 javadoc 9m 49s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 checkstyle 0m 29s There were no new checkstyle issues. +1 whitespace 0m 2s The patch has no lines that end in whitespace. +1 install 1m 31s mvn install still works. +1 eclipse:eclipse 0m 34s The patch built with eclipse:eclipse. +1 findbugs 1m 30s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 54m 46s Tests passed in hadoop-yarn-server-resourcemanager.     92m 17s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12753127/YARN-3970.20150829-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / e2c9b28 hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8942/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8942/testReport/ Java 1.7.0_55 uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/8942/console This message was automatically generated.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Updated the following :

        1. test case name modified to testUpdateAppPriority
        2. More information in the console output of Application CLI for updatePriority
        3. Formatting of the logged messages in CapacityScheduler
        4. Set proper error message in the response content when application is not in the desired state for updating the App's priority
        Show
        Naganarasimha Naganarasimha G R added a comment - Updated the following : test case name modified to testUpdateAppPriority More information in the console output of Application CLI for updatePriority Formatting of the logged messages in CapacityScheduler Set proper error message in the response content when application is not in the desired state for updating the App's priority
        Hide
        vvasudev Varun Vasudev added a comment -

        Couple of things -
        1. Can you add a GET method for the priority as well?
        2. The JSON produced converts the integer to a string({"priority":"8"}). You can fix this by adding the AppPriority class to org.apache.hadoop.yarn.server.resourcemanager.webapp.JAXBContextResolver.

        Show
        vvasudev Varun Vasudev added a comment - Couple of things - 1. Can you add a GET method for the priority as well? 2. The JSON produced converts the integer to a string({"priority":"8"}). You can fix this by adding the AppPriority class to org.apache.hadoop.yarn.server.resourcemanager.webapp.JAXBContextResolver.
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        0 pre-patch 18m 31s Pre-patch trunk compilation is healthy.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 9m 36s There were no new javac warning messages.
        +1 javadoc 13m 1s There were no new javadoc warning messages.
        +1 release audit 0m 31s The applied patch does not increase the total number of release audit warnings.
        -1 checkstyle 1m 36s The applied patch generated 3 new checkstyle issues (total was 164, now 167).
        +1 whitespace 0m 1s The patch has no lines that end in whitespace.
        +1 install 1m 43s mvn install still works.
        +1 eclipse:eclipse 0m 45s The patch built with eclipse:eclipse.
        +1 findbugs 3m 13s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 7m 14s Tests passed in hadoop-yarn-client.
        +1 yarn tests 58m 36s Tests passed in hadoop-yarn-server-resourcemanager.
            114m 50s  



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12753325/YARN-3970.20150831-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle
        git revision trunk / cf83156
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8952/testReport/
        Java 1.7.0_55
        uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8952/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment 0 pre-patch 18m 31s Pre-patch trunk compilation is healthy. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 9m 36s There were no new javac warning messages. +1 javadoc 13m 1s There were no new javadoc warning messages. +1 release audit 0m 31s The applied patch does not increase the total number of release audit warnings. -1 checkstyle 1m 36s The applied patch generated 3 new checkstyle issues (total was 164, now 167). +1 whitespace 0m 1s The patch has no lines that end in whitespace. +1 install 1m 43s mvn install still works. +1 eclipse:eclipse 0m 45s The patch built with eclipse:eclipse. +1 findbugs 3m 13s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 7m 14s Tests passed in hadoop-yarn-client. +1 yarn tests 58m 36s Tests passed in hadoop-yarn-server-resourcemanager.     114m 50s   Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12753325/YARN-3970.20150831-1.patch Optional Tests javadoc javac unit findbugs checkstyle git revision trunk / cf83156 checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/testrun_hadoop-yarn-client.txt hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8952/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8952/testReport/ Java 1.7.0_55 uname Linux asf907.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/8952/console This message was automatically generated.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks Varun Vasudev, for the review.

        Can you add a GET method for the priority as well?

        Application's Priority is currently available as part of the application report (/apps or /apps/{appid}) , explicit separate API for getting Application Priority is req ?

        The JSON produced converts the integer to a string({"priority":"8"})

        Adding it to JAXBContextResolver.rootUnWrappedTypes worked as you suggested, but wanted to know when to put to rootUnWrappedTypes and cTypes i.e. when rootUnwrapping needs to be set to true and when to false?

        Only 3rd checkstyle issue is valid and once Varun Vasudev confirms, will provide the corrections in next patch

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks Varun Vasudev , for the review. Can you add a GET method for the priority as well? Application's Priority is currently available as part of the application report ( /apps or /apps/{appid }) , explicit separate API for getting Application Priority is req ? The JSON produced converts the integer to a string({"priority":"8"}) Adding it to JAXBContextResolver.rootUnWrappedTypes worked as you suggested, but wanted to know when to put to rootUnWrappedTypes and cTypes i.e. when rootUnwrapping needs to be set to true and when to false? Only 3rd checkstyle issue is valid and once Varun Vasudev confirms, will provide the corrections in next patch
        Hide
        sunilg Sunil G added a comment -

        Hi Naganarasimha G R
        Thanks for the updated patch.

        One more point.
        XmlRootElement(name = "apppriority")
        In AppPriority, I feel root name can be applicationpriority. Current one may cause typo pblms in real use cases.

        Show
        sunilg Sunil G added a comment - Hi Naganarasimha G R Thanks for the updated patch. One more point. XmlRootElement(name = "apppriority") In AppPriority, I feel root name can be applicationpriority . Current one may cause typo pblms in real use cases.
        Hide
        vvasudev Varun Vasudev added a comment -

        Application's Priority is currently available as part of the application report (/apps or /apps/{appid}) , explicit separate API for getting Application Priority is req ?

        It's useful to have.

        when rootUnwrapping needs to be set to true and when to false?

        If you want the AppPriority dao JSON to be { "apppriority": { "priority": 8 } }, set root unwrapping to be false. If you want it to be { "priority": 8}, set it to true.

        I also forgot to mention, please update the web services documentation providing examples of how to use the REST service.

        Show
        vvasudev Varun Vasudev added a comment - Application's Priority is currently available as part of the application report (/apps or /apps/{appid}) , explicit separate API for getting Application Priority is req ? It's useful to have. when rootUnwrapping needs to be set to true and when to false? If you want the AppPriority dao JSON to be { "apppriority": { "priority": 8 } }, set root unwrapping to be false. If you want it to be { "priority": 8}, set it to true. I also forgot to mention, please update the web services documentation providing examples of how to use the REST service.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks for the comments Varun Vasudev,

        It's useful to have.

        It would be good to know the scenario too, IMHO the Application Report got from the REST call also is not bulky and currently we do not have any rest calls to get individual items of the Application report like the queue name which it has been submitted to, or the node label expression etc.. so would it help to have separate REST for priority only ? also if REST is required, would it be req in the case of CLI too ?

        If you want the AppPriority dao JSON to be { "apppriority": { "priority": 8 } }, set root unwrapping to be false.

        Well realized this while testing but my query was more like when we require root unwrapping to be true and when to be false ?

        Will handle the WebServices Documentation update too as part of this patch.

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks for the comments Varun Vasudev , It's useful to have. It would be good to know the scenario too, IMHO the Application Report got from the REST call also is not bulky and currently we do not have any rest calls to get individual items of the Application report like the queue name which it has been submitted to, or the node label expression etc.. so would it help to have separate REST for priority only ? also if REST is required, would it be req in the case of CLI too ? If you want the AppPriority dao JSON to be { "apppriority": { "priority": 8 } }, set root unwrapping to be false. Well realized this while testing but my query was more like when we require root unwrapping to be true and when to be false ? Will handle the WebServices Documentation update too as part of this patch.
        Hide
        vvasudev Varun Vasudev added a comment -

        It would be good to know the scenario too, IMHO the Application Report got from the REST call also is not bulky and currently we do not have any rest calls to get individual items of the Application report like the queue name which it has been submitted to, or the node label expression etc.. so would it help to have separate REST for priority only ? also if REST is required, would it be req in the case of CLI too ?

        It's useful when you use the web service to change the priority and want to confirm that it got changed. The equivalent GET call exists for the app queue and it should probably exist for the node label expression as well.

        Show
        vvasudev Varun Vasudev added a comment - It would be good to know the scenario too, IMHO the Application Report got from the REST call also is not bulky and currently we do not have any rest calls to get individual items of the Application report like the queue name which it has been submitted to, or the node label expression etc.. so would it help to have separate REST for priority only ? also if REST is required, would it be req in the case of CLI too ? It's useful when you use the web service to change the priority and want to confirm that it got changed. The equivalent GET call exists for the app queue and it should probably exist for the node label expression as well.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks Varun Vasudev & Sunil G,
        I have updated a new patch with fixes for the comments given . Please take a look

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks Varun Vasudev & Sunil G , I have updated a new patch with fixes for the comments given . Please take a look
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        -1 pre-patch 20m 38s Pre-patch trunk has 1 extant Findbugs (version 3.0.0) warnings.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 7m 45s There were no new javac warning messages.
        +1 javadoc 10m 5s There were no new javadoc warning messages.
        +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings.
        +1 site 3m 4s Site still builds.
        -1 checkstyle 1m 21s The applied patch generated 4 new checkstyle issues (total was 168, now 172).
        +1 whitespace 0m 6s The patch has no lines that end in whitespace.
        +1 install 1m 30s mvn install still works.
        +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
        +1 findbugs 2m 24s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        +1 yarn tests 6m 55s Tests passed in hadoop-yarn-client.
        -1 yarn tests 54m 19s Tests failed in hadoop-yarn-server-resourcemanager.
            109m 9s  



        Reason Tests
        Failed unit tests hadoop.yarn.server.resourcemanager.rmapp.TestRMAppTransitions



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12753515/YARN-3970.20150901-1.patch
        Optional Tests javadoc javac unit findbugs checkstyle site
        git revision trunk / faa38e1
        Pre-patch Findbugs warnings https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/trunkFindbugsWarningshadoop-yarn-server-resourcemanager.html
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8958/testReport/
        Java 1.7.0_55
        uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8958/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment -1 pre-patch 20m 38s Pre-patch trunk has 1 extant Findbugs (version 3.0.0) warnings. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 7m 45s There were no new javac warning messages. +1 javadoc 10m 5s There were no new javadoc warning messages. +1 release audit 0m 24s The applied patch does not increase the total number of release audit warnings. +1 site 3m 4s Site still builds. -1 checkstyle 1m 21s The applied patch generated 4 new checkstyle issues (total was 168, now 172). +1 whitespace 0m 6s The patch has no lines that end in whitespace. +1 install 1m 30s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 2m 24s The patch does not introduce any new Findbugs (version 3.0.0) warnings. +1 yarn tests 6m 55s Tests passed in hadoop-yarn-client. -1 yarn tests 54m 19s Tests failed in hadoop-yarn-server-resourcemanager.     109m 9s   Reason Tests Failed unit tests hadoop.yarn.server.resourcemanager.rmapp.TestRMAppTransitions Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12753515/YARN-3970.20150901-1.patch Optional Tests javadoc javac unit findbugs checkstyle site git revision trunk / faa38e1 Pre-patch Findbugs warnings https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/trunkFindbugsWarningshadoop-yarn-server-resourcemanager.html checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/testrun_hadoop-yarn-client.txt hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8958/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8958/testReport/ Java 1.7.0_55 uname Linux asf906.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/8958/console This message was automatically generated.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        correcting some checkstyle issues and test case failure seems to be unrelated to this jira.

        Show
        Naganarasimha Naganarasimha G R added a comment - correcting some checkstyle issues and test case failure seems to be unrelated to this jira.
        Hide
        hadoopqa Hadoop QA added a comment -



        -1 overall



        Vote Subsystem Runtime Comment
        -1 pre-patch 21m 4s Pre-patch trunk has 1 extant Findbugs (version 3.0.0) warnings.
        +1 @author 0m 0s The patch does not contain any @author tags.
        +1 tests included 0m 0s The patch appears to include 1 new or modified test files.
        +1 javac 8m 0s There were no new javac warning messages.
        +1 javadoc 10m 5s There were no new javadoc warning messages.
        +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings.
        +1 site 3m 4s Site still builds.
        -1 checkstyle 1m 26s The applied patch generated 2 new checkstyle issues (total was 168, now 170).
        +1 whitespace 0m 4s The patch has no lines that end in whitespace.
        +1 install 1m 28s mvn install still works.
        +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse.
        +1 findbugs 2m 24s The patch does not introduce any new Findbugs (version 3.0.0) warnings.
        -1 yarn tests 6m 57s Tests failed in hadoop-yarn-client.
        +1 yarn tests 54m 39s Tests passed in hadoop-yarn-server-resourcemanager.
            110m 13s  



        Reason Tests
        Failed unit tests hadoop.yarn.client.api.impl.TestYarnClient



        Subsystem Report/Notes
        Patch URL http://issues.apache.org/jira/secure/attachment/12753532/YARN-3970.20150901-2.patch
        Optional Tests javadoc javac unit findbugs checkstyle site
        git revision trunk / 2e251a7
        Pre-patch Findbugs warnings https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/trunkFindbugsWarningshadoop-yarn-server-resourcemanager.html
        checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt
        hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/testrun_hadoop-yarn-client.txt
        hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt
        Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8959/testReport/
        Java 1.7.0_55
        uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux
        Console output https://builds.apache.org/job/PreCommit-YARN-Build/8959/console

        This message was automatically generated.

        Show
        hadoopqa Hadoop QA added a comment - -1 overall Vote Subsystem Runtime Comment -1 pre-patch 21m 4s Pre-patch trunk has 1 extant Findbugs (version 3.0.0) warnings. +1 @author 0m 0s The patch does not contain any @author tags. +1 tests included 0m 0s The patch appears to include 1 new or modified test files. +1 javac 8m 0s There were no new javac warning messages. +1 javadoc 10m 5s There were no new javadoc warning messages. +1 release audit 0m 23s The applied patch does not increase the total number of release audit warnings. +1 site 3m 4s Site still builds. -1 checkstyle 1m 26s The applied patch generated 2 new checkstyle issues (total was 168, now 170). +1 whitespace 0m 4s The patch has no lines that end in whitespace. +1 install 1m 28s mvn install still works. +1 eclipse:eclipse 0m 33s The patch built with eclipse:eclipse. +1 findbugs 2m 24s The patch does not introduce any new Findbugs (version 3.0.0) warnings. -1 yarn tests 6m 57s Tests failed in hadoop-yarn-client. +1 yarn tests 54m 39s Tests passed in hadoop-yarn-server-resourcemanager.     110m 13s   Reason Tests Failed unit tests hadoop.yarn.client.api.impl.TestYarnClient Subsystem Report/Notes Patch URL http://issues.apache.org/jira/secure/attachment/12753532/YARN-3970.20150901-2.patch Optional Tests javadoc javac unit findbugs checkstyle site git revision trunk / 2e251a7 Pre-patch Findbugs warnings https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/trunkFindbugsWarningshadoop-yarn-server-resourcemanager.html checkstyle https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/diffcheckstylehadoop-yarn-server-resourcemanager.txt hadoop-yarn-client test log https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/testrun_hadoop-yarn-client.txt hadoop-yarn-server-resourcemanager test log https://builds.apache.org/job/PreCommit-YARN-Build/8959/artifact/patchprocess/testrun_hadoop-yarn-server-resourcemanager.txt Test Results https://builds.apache.org/job/PreCommit-YARN-Build/8959/testReport/ Java 1.7.0_55 uname Linux asf905.gq1.ygridcore.net 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux Console output https://builds.apache.org/job/PreCommit-YARN-Build/8959/console This message was automatically generated.
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Seems like test case failure is not related to this patch. Varun Vasudev, Rohith Sharma K S & Sunil G, can any of you take a look at the latest patch.

        Show
        Naganarasimha Naganarasimha G R added a comment - Seems like test case failure is not related to this patch. Varun Vasudev , Rohith Sharma K S & Sunil G , can any of you take a look at the latest patch.
        Hide
        vvasudev Varun Vasudev added a comment -

        +1 for the latest patch. I'll commit this tomorrow if no one objects.

        Show
        vvasudev Varun Vasudev added a comment - +1 for the latest patch. I'll commit this tomorrow if no one objects.
        Hide
        vvasudev Varun Vasudev added a comment -

        Committed to trunk and branch-2. Thanks Naganarasimha G R!

        Show
        vvasudev Varun Vasudev added a comment - Committed to trunk and branch-2. Thanks Naganarasimha G R !
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-trunk-Commit #8394 (See https://builds.apache.org/job/Hadoop-trunk-Commit/8394/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-trunk-Commit #8394 (See https://builds.apache.org/job/Hadoop-trunk-Commit/8394/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #339 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/339/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk-Java8 #339 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Java8/339/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #346 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/346/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk-Java8 #346 (See https://builds.apache.org/job/Hadoop-Yarn-trunk-Java8/346/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        Hide
        Naganarasimha Naganarasimha G R added a comment -

        Thanks for review and commit Varun Vasudev, Sunil G & Rohith Sharma K S

        Show
        Naganarasimha Naganarasimha G R added a comment - Thanks for review and commit Varun Vasudev , Sunil G & Rohith Sharma K S
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Yarn-trunk #1076 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/1076/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Yarn-trunk #1076 (See https://builds.apache.org/job/Hadoop-Yarn-trunk/1076/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Mapreduce-trunk #2288 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2288/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Mapreduce-trunk #2288 (See https://builds.apache.org/job/Hadoop-Mapreduce-trunk/2288/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #329 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/329/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk-Java8 #329 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk-Java8/329/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Hadoop-Hdfs-trunk #2267 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2267/)
        YARN-3970. Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163)

        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md
        • hadoop-yarn-project/CHANGES.txt
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java
        • hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Hadoop-Hdfs-trunk #2267 (See https://builds.apache.org/job/Hadoop-Hdfs-trunk/2267/ ) YARN-3970 . Add REST api support for Application Priority. Contributed by Naganarasimha G R. (vvasudev: rev b469ac531af1bdda01a04ae0b8d39218ca292163) hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/dao/AppPriority.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/JAXBContextResolver.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-site/src/site/markdown/ResourceManagerRest.md hadoop-yarn-project/CHANGES.txt hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/CapacityScheduler.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/RMWebServices.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/webapp/TestRMWebServicesAppsModification.java hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/cli/ApplicationCLI.java

          People

          • Assignee:
            Naganarasimha Naganarasimha G R
            Reporter:
            sunilg Sunil G
          • Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development