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: 

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.


> 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

Reply via email to