[ 
https://issues.apache.org/jira/browse/YARN-4023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15249116#comment-15249116
 ] 

Sunil G commented on YARN-4023:
-------------------------------

HI [~jianhe],

I have checked below code in {{RMAppManager.createAndPopulateNewRMApp}}. With 
this piece of code, even if priority is NOT set in submissionContext, we will 
set it with a default priority for any apps (with 0 priority in dflt case). 
This will not impact any apps untill we enable app priority from top level.

{code}
    // Verify and get the update application priority and set back to
    // submissionContext
    Priority appPriority = rmContext.getScheduler()
        .checkAndGetApplicationPriority(submissionContext.getPriority(), user,
            submissionContext.getQueue(), applicationId);
    submissionContext.setPriority(appPriority);
{code}

This same code is invoked from {{recoverApplication}} also. So even in rolling 
upgrade cases where app is saved to Store w/o priority in submission context, 
it will be added wth a priority 0 with above piece of code during recovery. 
[~jianhe] will  this be fine? or do you see some other problem because of this. 
pls share your feedback.



> Publish Application Priority to TimelineServer
> ----------------------------------------------
>
>                 Key: YARN-4023
>                 URL: https://issues.apache.org/jira/browse/YARN-4023
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: timelineserver
>            Reporter: Sunil G
>            Assignee: Sunil G
>             Fix For: 2.8.0
>
>         Attachments: 0001-YARN-4023.patch, 0001-YARN-4023.patch, 
> ApplicationPage.png, TimelineserverMainpage.png
>
>
> Publish Application priority details to Timeline Server. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to