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

Chris Nauroth commented on MAPREDUCE-4869:
------------------------------------------

It might help to view this test side-by-side with the code in 
{{MapReduceChildJVM#getVMCommand}}.  That method is responsible for creating 
the command to launch in the YARN container.  The method uses 
{{Environment.JAVA_HOME.$()}} and {{Environment.PWD.$()}}, which translate the 
environment variables to $JAVA_HOME and $PWD or %JAVA_HOME% and %PWD% depending 
on the platform.

Great catch on %CD% vs. %PWD%.  We missed that in YARN-233, and it's probably a 
problem.  I think jobs are still working right now, because it's used in 
java.io.tmpdir=%PWD%/tmp, and the %PWD% resolves to an empty string, so it just 
uses /tmp.  It works, but it isn't really correct.  Do you mind if I spin off a 
separate jira to address this?  I think the scope of that change is going to be 
much larger.

                
> TestMapReduceChildJVM fails in branch-trunk-win
> -----------------------------------------------
>
>                 Key: MAPREDUCE-4869
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4869
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: test
>    Affects Versions: trunk-win
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>         Attachments: MAPREDUCE-4869-branch-trunk-win.1.patch
>
>
> The YARN-233 patch for getting YARN working on Windows forgot to include a 
> corresponding change in {{TestMapReduceChildJVM}}, so the test is failing now.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to