[ 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