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

Hudson commented on HADOOP-16351:
---------------------------------

FAILURE: Integrated in Jenkins build Hadoop-trunk-Commit #17138 (See 
[https://builds.apache.org/job/Hadoop-trunk-Commit/17138/])
HADOOP-16351. Change ":" to ApplicationConstants.CLASS_PATH_SEPARATOR. 
(weichiu: rev 8d754c2c397f1273ad7c520fa5f1b7efa0db31f7)
* (edit) 
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-applications/hadoop-yarn-applications-distributedshell/src/main/java/org/apache/hadoop/yarn/applications/distributedshell/Client.java


> Change ":" to ApplicationConstants.CLASS_PATH_SEPARATOR
> -------------------------------------------------------
>
>                 Key: HADOOP-16351
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16351
>             Project: Hadoop Common
>          Issue Type: Task
>          Components: common
>    Affects Versions: 3.1.2
>            Reporter: kevin su
>            Assignee: kevin su
>            Priority: Trivial
>             Fix For: 3.3.0
>
>         Attachments: HADOOP-16351.01.patch
>
>
> under distributedshell/Clients.java 
> We should change ":" to ApplicationConstants.CLASS_PATH_SEPARATOR, so it 
> could also support Windows client
> {code}
> // add the runtime classpath needed for tests to work
>     if (conf.getBoolean(YarnConfiguration.IS_MINI_YARN_CLUSTER, false)) {
>       classPathEnv.append(':')
>           .append(System.getProperty("java.class.path"));
>     }
> {code}
> {code}
> // add the runtime classpath needed for tests to work
>     if (conf.getBoolean(YarnConfiguration.IS_MINI_YARN_CLUSTER, false)) {
>       classPathEnv.append(ApplicationConstants.CLASS_PATH_SEPARATOR)
>           .append(System.getProperty("java.class.path"));
>     }
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.14#76016)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to