[ 
https://issues.apache.org/jira/browse/YARN-2827?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14202455#comment-14202455
 ] 

Vinod Kumar Vavilapalli commented on YARN-2827:
-----------------------------------------------

I had reviewed this at YARN-2647 which caused this.

The patch looks good, +1.

Will resubmit the trunk patch to Jenkins and then commit..

> Fix bugs of yarn queue CLI
> --------------------------
>
>                 Key: YARN-2827
>                 URL: https://issues.apache.org/jira/browse/YARN-2827
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: client
>            Reporter: Wangda Tan
>            Assignee: Wangda Tan
>            Priority: Critical
>         Attachments: YARN-2827-branch_2-1.patch, YARN-2827.1.patch, 
> YARN-2827.1.patch
>
>
> Need fix bugs:
> 1) args of queue CLI is without "queue" even if you run with "yarn queue 
> -status ..", the args is ["-status", "..."]. The assumption is incorrect.
> 2) It is possible that there's no related QueueInfo with specified queue 
> name, and null will be returned from YarnClient, so NPE will raise. Added a 
> check for it, and will print proper message
> 3) When failed to get QueueInfo, should return non-zero exit code.
> 4) Add tests for above.



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

Reply via email to