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

Suma Shivaprasad edited comment on YARN-7586 at 12/2/17 4:52 AM:
-----------------------------------------------------------------

Thanks [~sunilg] Attached patch with comments addressed.

RMActiveServiceContext ensures that PlacementManager is not null since is it 
created during its construction. This is just a saftey check to make sure its 
set. Have added an additional check to log the error and throw an exception if 
the specified queue is also invalid



was (Author: suma.shivaprasad):
Thanks [~sunilg] Attached patch with comments addressed.

RMActiveServiceContext ensures that PlacementManager is no null since is it 
created during its construction. This is just a saftey check to make sure its 
set. Have added an additional check to log the error and throw an exception if 
the specified queue is also invalid


> Application Placement should be done before ACL checks in ResourceManager
> -------------------------------------------------------------------------
>
>                 Key: YARN-7586
>                 URL: https://issues.apache.org/jira/browse/YARN-7586
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Suma Shivaprasad
>            Assignee: Suma Shivaprasad
>            Priority: Blocker
>         Attachments: YARN-7586.1.patch, YARN-7586.2.patch, YARN-7586.3.patch, 
> YARN-7586.4.patch
>
>
> YARN-7419 moved Application placement from RMAppManager to RMAppImpl which 
> causes issues since ApplicationSubmissionContext still has the original queue 
> specified by the user and not the mapped queue . This causes issues while 
> doing ACL checks in RMAppManager



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to