[ 
https://issues.apache.org/jira/browse/YARN-233?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Chris Nauroth updated YARN-233:
-------------------------------

    Attachment: YARN-233-branch-trunk-win.3.patch

Thanks again, Bikas.  I've attached version 3 of the patch.  I now have 
separate methods for containerExists and killContainer.  To prevent the need to 
change the native code, I've created a subclass of Shell called ProcessIsAlive 
to handle (and hide) the details of "winutils task isAlive" vs. "kill -0" and 
their different error handling strategies (stdout message vs. non-zero exit 
code).  Regarding ProcessIdFileReader, I've restored the existing pid 
validation logic and also added Windows-specific logic for validating that the 
pid is a container ID.  I've updated TestProcessIdFileReader too and verified 
that it passes on Mac and Windows.
                
> YARN support for execution of container on Windows
> --------------------------------------------------
>
>                 Key: YARN-233
>                 URL: https://issues.apache.org/jira/browse/YARN-233
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: nodemanager
>    Affects Versions: trunk-win
>            Reporter: Chris Nauroth
>            Assignee: Chris Nauroth
>         Attachments: YARN-233-branch-trunk-win.3.patch, 
> YARN-233-branch-trunk-win.patch, YARN-233-branch-trunk-win.patch
>
>
> YARN container launch code requires updates for compatibility with Windows.  
> Instead of launching and killing tasks via generated bash scripts and kill, 
> use generated cmd scripts and winutils for task management.

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