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

Xuan Gong commented on YARN-1303:
---------------------------------

bq.please remove. I think it is intuitive. Similarly the exception code message

Removed

bq. This will not work on windows. 

For windows, it support 
{code}
The conjunctions are:
&
The simplest conjunction. The next pipeline is always executed after the 
current one has completed executing.
&&
A positive conditional conjunction. The next pipeline is executed if the 
current one completes executing with a zero exit status.
||
A negative conditional conjunction. The next pipeline is executed if the 
current one completes executing with a non-zero exit status.
{code} 

So, add checking for '&'

bq.Btw do we really need to parse and tell user that you are indeed using 
multiple commands instead of allowed one? Now we are putting this in help 
message I think this validation checks will complicate the stuff. thoughts?

I think we can add those validation checks should be fine although we have 
already mentioned those in the help message. If the clients do add these 
"special" character into their command, we can warm them as early as possible 
instead of running those unsupported commands to get the unexpected results 

> Allow multiple commands separating with ";" in distributed-shell
> ----------------------------------------------------------------
>
>                 Key: YARN-1303
>                 URL: https://issues.apache.org/jira/browse/YARN-1303
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: applications/distributed-shell
>            Reporter: Tassapol Athiapinya
>            Assignee: Xuan Gong
>             Fix For: 2.2.1
>
>         Attachments: YARN-1303.1.patch, YARN-1303.2.patch, YARN-1303.3.patch, 
> YARN-1303.3.patch, YARN-1303.4.patch, YARN-1303.4.patch, YARN-1303.5.patch, 
> YARN-1303.6.patch
>
>
> In shell, we can do "ls; ls" to run 2 commands at once. 
> In distributed shell, this is not working. We should improve to allow this to 
> occur. There are practical use cases that I know of to run multiple commands 
> or to set environment variables before a command.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to