Karthik Kambatla commented on YARN-2130:

Like the direction this is going. I think we can go a step further and do a 
couple of other things:
# Get rid of getters in ResourceManager corresponding to those added to 
RMContext in this patch. 
# Optionally, get rid of the fields that cache these objects. IMO, we should 
move towards having to pass only RMContext while creating child services.
# It has nothing to do with this patch, but there are other similar items with 
duplicate getters in ResourceManager and RMContext. It would be nice to get rid 
of the RM methods for those too.

[~ozawa], [~vinodkv] - do you guys agree that would clean things up more? 

> Cleanup: Adding getRMAppManager, getQueueACLsManager, 
> getApplicationACLsManager to RMContext
> --------------------------------------------------------------------------------------------
>                 Key: YARN-2130
>                 URL: https://issues.apache.org/jira/browse/YARN-2130
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Tsuyoshi OZAWA
>            Assignee: Tsuyoshi OZAWA
>         Attachments: YARN-2130.1.patch

This message was sent by Atlassian JIRA

Reply via email to