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

Thank You [~leftnoteasy] for the detailed comments.

I have updated patch as per same. 

Few notes:
1. NodeLabelsName and NodeToLabelsName classes are needed for replace and 
remove apis. Else as you mentioned, REST apis wont be clean enough. I hope this 
is fine.
bq.getLabelsToNodes should use object
>> Done
bq.replace/remove should use list of label name only
>> Done

> 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
> 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