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

Chris Nauroth commented on YARN-593:
------------------------------------

I see it now.  I missed this.  Thanks!  I'll put together a new patch that 
checks isDirectory just once in the outer loop.
                
> container launch on Windows does not correctly populate classpath with new 
> process's environment variables and localized resources
> ----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-593
>                 URL: https://issues.apache.org/jira/browse/YARN-593
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>    Affects Versions: 3.0.0
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>         Attachments: YARN-593.1.patch
>
>
> On Windows, we must bundle the classpath of a launched container in an 
> intermediate jar with a manifest.  Currently, this logic incorrectly uses the 
> nodemanager process's environment variables for substitution.  Instead, it 
> needs to use the new environment for the launched process.  Also, the bundled 
> classpath is missing some localized resources for directories, due to a quirk 
> in the way {{File#toURI}} decides whether or not to append a trailing '/'.

--
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