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

Bilwa S T commented on YARN-9459:
---------------------------------

Hi [~snemeth]
ConfiguredYarnAuthorizer#checkPermission is not just called from 
QueueACLsManager. its called from RMAppManager and even Scheduler. If queue 
related code is removed from here, then we need to make changes from all 
methods.

> ConfiguredYarnAuthorizer should not now any details about queues
> ----------------------------------------------------------------
>
>                 Key: YARN-9459
>                 URL: https://issues.apache.org/jira/browse/YARN-9459
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Assignee: Bilwa S T
>            Priority: Major
>
> ConfiguredYarnAuthorizer should not now anything about queues, as this is not 
> its responsibility, see 
> [here|https://github.com/apache/hadoop/blob/39b4a37e02e929a698fcf9e32f1f71bb6b977635/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-common/src/main/java/org/apache/hadoop/yarn/security/ConfiguredYarnAuthorizer.java#L70]
> Code like this could be part of the QueueACLsManager.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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

Reply via email to