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

Sunil G commented on YARN-2301:
-------------------------------

This will be really useful enhancement.

I have a concern here. 
bq. yarn container -list <appId|appAttemptId> <additional opts>

* *list* with *<additional opts>* comes after the variable input from user 
(appId|appAttemptId).
And <additional opts> is only for one of the type named *appId*. May be it may 
confuse user also, like which sub option needs the <additional opts>. 
I feel may be we can have a new command itself for listing application 
container.
A suggestion is:
{noformat}
yarn container -list-appid <appId> <additional opts> 
yarn container -list-appattemptid  <appAttemptId>
{noformat}
OR
{noformat}
yarn application -list-containers <appId>
{noformat}

* I feel sequential checks with ConverterUtils.toApplicationID and 
ConverterUtils.toApplicationAttemptId has to be done to know whether input is 
appId|appAttemptId.
So rediercting to my point 1, if seperate command is there, may be it can be 
handled in a better way from applicationCLI (rather than handling specific 
types of exceptions).
Please share your thoughts

> Improve yarn container command
> ------------------------------
>
>                 Key: YARN-2301
>                 URL: https://issues.apache.org/jira/browse/YARN-2301
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Jian He
>            Assignee: Naganarasimha G R
>              Labels: usability
>
> While running yarn container -list <Application Attempt ID> command, some 
> observations:
> 1) the scheme (e.g. http/https  ) before LOG-URL is missing
> 2) the start-time is printed as milli seconds (e.g. 1405540544844). Better to 
> print as time format.
> 3) finish-time is 0 if container is not yet finished. May be "N/A"
> 4) May have an option to run as yarn container -list <appId> OR  yarn 
> application -list-containers <appId> also.  
> As attempt Id is not shown on console, this is easier for user to just copy 
> the appId and run it, may  also be useful for container-preserving AM 
> restart. 



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to