[
https://issues.apache.org/jira/browse/HADOOP-2721?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12657694#action_12657694
]
Vinod K V commented on HADOOP-2721:
-----------------------------------
We should document on which platforms we have setsid available. If it isn't
availabe everywhere, we should in the minimum try to fall back to the current
mechanism of launching tasks without job control.
Few more code comments:
- destroySession() also goes beyond the scope of ProcfsBasedProcessTree, it
should be moved out.
- Add a test-case to test that SIGKILL works.
- I think assertions are disabled by default, instead you should have explicit
checks.
> Use job control for tasks (and therefore for pipes and streaming)
> -----------------------------------------------------------------
>
> Key: HADOOP-2721
> URL: https://issues.apache.org/jira/browse/HADOOP-2721
> Project: Hadoop Core
> Issue Type: Improvement
> Components: contrib/streaming, mapred, pipes
> Reporter: Owen O'Malley
> Assignee: Ravi Gummadi
> Attachments: HADOOP-2721.patch, HADOOP-2721.patch, HADOOP-2721.patch
>
>
> We should use the setsid command when the task is launched to create a new
> session. We should be able to use the setsid program when we launch the bash
> process to create a new session. That will allow us to kill the entire
> session with a single signal and remove the need for the ping methods in both
> TaskTracker.Child and pipes.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
