[ https://issues.apache.org/jira/browse/YARN-1666?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13910888#comment-13910888 ]
Hitesh Shah commented on YARN-1666: ----------------------------------- [~xgong] Those newly added files are in the wrong location. [~vinodkv] In any case, the above committed patch seems a bit wrong to me. If someone is using a Configuration object with loaded resources, say core-site, yarn-site and foo-site followed by some Configuration::set() calls, the above code will override all conflicting settings. This seems wrong especially in the MiniYARNCluster case. > Make admin refreshNodes work across RM failover > ----------------------------------------------- > > Key: YARN-1666 > URL: https://issues.apache.org/jira/browse/YARN-1666 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Xuan Gong > Assignee: Xuan Gong > Fix For: 2.4.0 > > Attachments: YARN-1666.1.patch, YARN-1666.2.patch, YARN-1666.2.patch, > YARN-1666.3.patch, YARN-1666.4.patch, YARN-1666.4.patch, YARN-1666.5.patch, > YARN-1666.6.patch > > -- This message was sent by Atlassian JIRA (v6.1.5#6160)