Hadoop Common
  1. Hadoop Common
  2. HADOOP-7722

Support a stop command in ShellCommandExecutor

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Later
    • Affects Version/s: 0.23.0
    • Fix Version/s: 0.23.0
    • Component/s: None
    • Labels:
      None
    • Tags:
      mr2,mapreduce-2.0

      Description

      Add a stop function call to Shell so as to either stop the process from getting started if not already done so or destroy the process if it already has started.

      Context of usage:
      There can be race conditions when trying to kill an execution as the shell::execute call is blocking and using the getProcess call is not a feasible option as there can be a delay between the execute call and when the process object is actually created ( after which it can be used to kill the running sub-process ).

        Issue Links

          Activity

          Hitesh Shah created issue -
          Hide
          Hitesh Shah added a comment -

          Added a stop function that allows someone to stop the execution of the shell mid-way.

          Show
          Hitesh Shah added a comment - Added a stop function that allows someone to stop the execution of the shell mid-way.
          Hitesh Shah made changes -
          Field Original Value New Value
          Attachment HADOOP-7722.1.patch [ 12497900 ]
          Hitesh Shah made changes -
          Status Open [ 1 ] Patch Available [ 10002 ]
          Hide
          Hadoop QA added a comment -

          +1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12497900/HADOOP-7722.1.patch
          against trunk revision .

          +1 @author. The patch does not contain any @author tags.

          +1 tests included. The patch appears to include 3 new or modified tests.

          +1 javadoc. The javadoc tool did not generate any warning messages.

          +1 javac. The applied patch does not increase the total number of javac compiler warnings.

          +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings.

          +1 release audit. The applied patch does not increase the total number of release audit warnings.

          +1 core tests. The patch passed unit tests in .

          +1 contrib tests. The patch passed contrib unit tests.

          Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/268//testReport/
          Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/268//console

          This message is automatically generated.

          Show
          Hadoop QA added a comment - +1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12497900/HADOOP-7722.1.patch against trunk revision . +1 @author. The patch does not contain any @author tags. +1 tests included. The patch appears to include 3 new or modified tests. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs (version 1.3.9) warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed unit tests in . +1 contrib tests. The patch passed contrib unit tests. Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/268//testReport/ Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/268//console This message is automatically generated.
          Hitesh Shah made changes -
          Link This issue blocks MAPREDUCE-3084 [ MAPREDUCE-3084 ]
          Hide
          Vinod Kumar Vavilapalli added a comment -

          We don't need this anymore after the MAPREDUCE-3240 patch.

          Closing this for later, the patch is still useful for others who use ShellCommandExecutor.

          Show
          Vinod Kumar Vavilapalli added a comment - We don't need this anymore after the MAPREDUCE-3240 patch. Closing this for later, the patch is still useful for others who use ShellCommandExecutor.
          Vinod Kumar Vavilapalli made changes -
          Status Patch Available [ 10002 ] Resolved [ 5 ]
          Assignee Hitesh Shah [ hitesh ]
          Resolution Later [ 7 ]
          Arun C Murthy made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Link This issue blocks MAPREDUCE-3084 [ MAPREDUCE-3084 ]
          Gavin made changes -
          Link This issue is depended upon by MAPREDUCE-3084 [ MAPREDUCE-3084 ]

            People

            • Assignee:
              Hitesh Shah
              Reporter:
              Hitesh Shah
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development