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

Jian He commented on YARN-2003:
-------------------------------

thanks for the updating ! some more comments:

bq. Yes. We are planning to check for acl's (priority acls) in this method
A bunch of other similar ACL checks are done asynchronously (see 
LeafQueue#submitApplication), though I think failing submit call synchronously 
is better. We should also, at some point, move all the checks in one place, 
rather than spreading across multiple places. 
- AbstractYarnScheduler#updateApplicationPriority can be removed. It can be 
added whenever it’s needed. 
- SchedulableEntity#getApplicationPriority -> SchedulableEntity#getPriority?
- authenticateApplicationPriority -> checkAppPriority ?
- RMAppManager#createAndPopulateNewRMApp : if user doesn’t have access to the 
default app priority acl, it’ll be ignored.

- The existing priority is using the semantics that lower integer means higher 
priority. Is it more intuitive to use higher integer to indicate higher 
priority? Otherwise,  below code is quite confusing to me - if priority is less 
than maxPriority, throw exception;
{code}
if (priority.compareTo(getMaxClusterLevelAppPriority()) < 0) {
  
throw new YarnException("Invalid priority as Queue: " + queueName
      + " 
cannot support more than priority '"
      + getMaxClusterLevelAppPriority() + 
"'");
}

{code}
 - Change in AppAddedSchedulerEvent - how about changing the construction to 
just passing the submissionContext reference ? 
- Add a test that LeafQueue#pendingApplications is honoring app Priority too ?

> Support for Application priority : Changes in RM and Capacity Scheduler
> -----------------------------------------------------------------------
>
>                 Key: YARN-2003
>                 URL: https://issues.apache.org/jira/browse/YARN-2003
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Sunil G
>            Assignee: Sunil G
>              Labels: BB2015-05-TBR
>         Attachments: 0001-YARN-2003.patch, 00010-YARN-2003.patch, 
> 0002-YARN-2003.patch, 0003-YARN-2003.patch, 0004-YARN-2003.patch, 
> 0005-YARN-2003.patch, 0006-YARN-2003.patch, 0007-YARN-2003.patch, 
> 0008-YARN-2003.patch, 0009-YARN-2003.patch, 0011-YARN-2003.patch, 
> 0012-YARN-2003.patch, 0013-YARN-2003.patch, 0014-YARN-2003.patch, 
> 0015-YARN-2003.patch, 0016-YARN-2003.patch, 0017-YARN-2003.patch
>
>
> AppAttemptAddedSchedulerEvent should be able to receive the Job Priority from 
> Submission Context and store.
> Later this can be used by Scheduler.



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

Reply via email to