[ 
https://issues.apache.org/jira/browse/YARN-2505?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14183742#comment-14183742
 ] 

Craig Welch commented on YARN-2505:
-----------------------------------

[~leftnoteasy] I appreciate your reviewing this change and I understand your 
perspective on this particular aspect of it - but I think that caching the node 
label manager here is premature optimization that adds unnecessary complexity 
and future risk where there isn't a good reason to do so.  You'll notice that 
other implementations in the service are taking the same approach and 
retrieving references as needed without caching.  I'm planning to leave it 
as/is in this respect.  

> Support get/add/remove/change labels in RM REST API
> ---------------------------------------------------
>
>                 Key: YARN-2505
>                 URL: https://issues.apache.org/jira/browse/YARN-2505
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Wangda Tan
>            Assignee: Craig Welch
>         Attachments: YARN-2505.1.patch, YARN-2505.3.patch, YARN-2505.4.patch, 
> YARN-2505.5.patch, YARN-2505.6.patch, YARN-2505.7.patch, YARN-2505.8.patch, 
> YARN-2505.9.patch, YARN-2505.patch
>
>




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

Reply via email to