[ 
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: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to