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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
