Craig Welch commented on YARN-2647:

[~sunilg] in the yarn file you can drop the (s) from information(s) ( 
information is singular and plural :-) )

In QueueCLI.listQueues I think it is safe to not check for a null queue list 
(not entirely sure), but in printQueueInfo I do think you need to check that 
the nodeLabels list is not null

Otherwise, +1 from me

> Add yarn queue CLI to get queue infos
> -------------------------------------
>                 Key: YARN-2647
>                 URL: https://issues.apache.org/jira/browse/YARN-2647
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: client
>            Reporter: Wangda Tan
>            Assignee: Sunil G
>         Attachments: 0001-YARN-2647.patch, 0002-YARN-2647.patch, 
> 0003-YARN-2647.patch, 0004-YARN-2647.patch, 0005-YARN-2647.patch, 
> 0006-YARN-2647.patch

This message was sent by Atlassian JIRA

Reply via email to