Sunil G updated YARN-3521:
    Attachment: 0006-YARN-3521.patch

Hi [~leftnoteasy] [~vinodkv]
Uploading a newer version of patch. Addressed all the comments from Wangda 
except the NodeToLabelsName and NodeLabelsName..

NodeToLabelsName  now contains a List of  NodeToLabelEntry, and I use this 
class as the parameter for replaceNodeLabels api.
I tried using  List of  NodeToLabelEntry w/o the wrapper class called 
NodeToLabelsName ,but some how i was not able to test this from test case.

Pls suggest you thoughts on using this outer wrapper class rather List of 

> Support return structured NodeLabel objects in REST API when call 
> getClusterNodeLabels
> --------------------------------------------------------------------------------------
>                 Key: YARN-3521
>                 URL: https://issues.apache.org/jira/browse/YARN-3521
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: api, client, resourcemanager
>            Reporter: Wangda Tan
>            Assignee: Sunil G
>         Attachments: 0001-YARN-3521.patch, 0002-YARN-3521.patch, 
> 0003-YARN-3521.patch, 0004-YARN-3521.patch, 0005-YARN-3521.patch, 
> 0006-YARN-3521.patch
> In YARN-3413, yarn cluster CLI returns NodeLabel instead of String, we should 
> make the same change in REST API side to make them consistency.

This message was sent by Atlassian JIRA

Reply via email to