[ 
https://issues.apache.org/jira/browse/YARN-2189?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Trezzo updated YARN-2189:
-------------------------------
    Attachment: YARN-2189-trunk-v6.patch

Thanks for the comments [~kasha]. Attached is V6 which addresses most of the 
comments. Diff between V5 and V6: 
https://github.com/ctrezzo/hadoop/commit/e8d47fb3e8cea03c4f3545571f1b2c9593f0574e

One thing that I didn't change is making SCMAdminProtocolService#checkAcls use 
RMServerUtils#verifyAccess. I started to do this, but then realized this would 
require the SharedCacheManager package to depend on the ResourceManager 
package. I started to move the verifyAccess method to yarn-server-common, but 
then realized that it uses the RMAuditLogger.

I could create a slightly more generic verifyAccess method in 
yarn-server-common and make both servers use that if you want. Let me know.

Thanks!

> Admin service for cache manager
> -------------------------------
>
>                 Key: YARN-2189
>                 URL: https://issues.apache.org/jira/browse/YARN-2189
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Chris Trezzo
>            Assignee: Chris Trezzo
>         Attachments: YARN-2189-trunk-v1.patch, YARN-2189-trunk-v2.patch, 
> YARN-2189-trunk-v3.patch, YARN-2189-trunk-v4.patch, YARN-2189-trunk-v5.patch, 
> YARN-2189-trunk-v6.patch
>
>
> Implement the admin service for the shared cache manager. This service is 
> responsible for handling administrative commands such as manually running a 
> cleaner task.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to