tomicooler edited a comment on pull request #3938: URL: https://github.com/apache/hadoop/pull/3938#issuecomment-1042794272
@9uapaw about the ACLManagers: ApplicationACLsManager CapacityQueueACLsManager extends QueueACLsManager ConfiguredYarnAuthorizer extends YarnAuthorizationProvider AppPriorityACLsManager ReservationsACLsManager There are already some, and they are all have some (serious) design problems. Since queues may not exists yet when we need to check ACLs for them, it's probably not a good idea to store the ACLs in the queues. Anyway, I don't think I have enough expertise on YARN to make big refactors in this context. Even if we want to it should be in a separate PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org