[
https://issues.apache.org/jira/browse/YARN-3164?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14318791#comment-14318791
]
Xuan Gong commented on YARN-3164:
---------------------------------
Patch looks good over all.
But, is this new function really necessary ?
{code}
+ protected void setErrOut(PrintStream errOut) {
+ this.errOut = errOut;
+ }
{code}
If this one is only for testing purpose, probably we do not need it.
Please take a look at TestRMAdminCli.testHelp().
> rmadmin command usage prints incorrect command name
> ---------------------------------------------------
>
> Key: YARN-3164
> URL: https://issues.apache.org/jira/browse/YARN-3164
> Project: Hadoop YARN
> Issue Type: Bug
> Components: resourcemanager
> Affects Versions: 2.6.0
> Reporter: Bibin A Chundatt
> Assignee: Bibin A Chundatt
> Priority: Minor
> Attachments: YARN-3164.1.patch, YARN-3164.2.patch
>
>
> /hadoop/bin>{color:red} ./yarn rmadmin -transitionToActive {color}
> transitionToActive: incorrect number of arguments
> Usage:{color:red} HAAdmin {color} [-transitionToActive <serviceId>
> [--forceactive]]
> >{color:red} ./yarn HAAdmin {color}
> Error: Could not find or load main class HAAdmin
> Expected it should be rmadmin
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
