[jira] [Updated] (YARN-2924) Node to labels mapping should not transfer to lowercase when adding from RMAdminCLI

2014-12-09 Thread Jian He (JIRA) 

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

Jian He updated YARN-2924:
--
Target Version/s: 2.7.0  (was: 2.7.0, 2.6.1)

> Node to labels mapping should not transfer to lowercase when adding from 
> RMAdminCLI
> ---
>
> Key: YARN-2924
> URL: https://issues.apache.org/jira/browse/YARN-2924
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: client
>Reporter: Wangda Tan
>Assignee: Wangda Tan
> Fix For: 2.7.0
>
> Attachments: YARN-2924.1.patch
>
>
> In existing implementation, when parsing node-to-labels mapping from 
> RMAdminCLI, it transferred all labels to lowercase:
> {code}
>   for (int i = 1; i < splits.length; i++) {
> if (!splits[i].trim().isEmpty()) {
>   map.get(nodeId).add(splits[i].trim().toLowerCase());
> }
>   }
> {code}
> That is not correct, we should fix that.



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

[jira] [Updated] (YARN-2924) Node to labels mapping should not transfer to lowercase when adding from RMAdminCLI

2014-12-04 Thread Wangda Tan (JIRA) 

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

Wangda Tan updated YARN-2924:
-
Attachment: YARN-2924.1.patch

Attached a fix for this, without this fix, modified test will fail.

> Node to labels mapping should not transfer to lowercase when adding from 
> RMAdminCLI
> ---
>
> Key: YARN-2924
> URL: https://issues.apache.org/jira/browse/YARN-2924
> Project: Hadoop YARN
>  Issue Type: Sub-task
>  Components: client
>Reporter: Wangda Tan
>Assignee: Wangda Tan
> Attachments: YARN-2924.1.patch
>
>
> In existing implementation, when parsing node-to-labels mapping from 
> RMAdminCLI, it transferred all labels to lowercase:
> {code}
>   for (int i = 1; i < splits.length; i++) {
> if (!splits[i].trim().isEmpty()) {
>   map.get(nodeId).add(splits[i].trim().toLowerCase());
> }
>   }
> {code}
> That is not correct, we should fix that.



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