You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by cw...@apache.org on 2015/02/02 18:43:11 UTC

svn commit: r1656527 - in /uima/sandbox/uima-ducc/trunk: uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java

Author: cwiklik
Date: Mon Feb  2 17:43:11 2015
New Revision: 1656527

URL: http://svn.apache.org/r1656527
Log:
UIMA-4214 Modified to fix JP status when it terminates due to Work item failure

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java
    uima/sandbox/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java?rev=1656527&r1=1656526&r2=1656527&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/ManagedProcess.java Mon Feb  2 17:43:11 2015
@@ -423,8 +423,14 @@ public class ManagedProcess implements P
 						getDuccProcess().setReasonForStoppingProcess(
 								errors.trim());
 					} else if ( !isAP ){
-						getDuccProcess().setReasonForStoppingProcess(
-								ReasonForStoppingProcess.Croaked.toString());
+						if ( exitcode == 0 || exitcode == 128) {
+							pstate = ProcessState.Stopped;
+							
+						} else {
+							getDuccProcess().setReasonForStoppingProcess(
+									ReasonForStoppingProcess.Croaked.toString());
+							
+						}
 					}
 				} else if ( getDuccProcess().getProcessState().equals(ProcessState.Stopping)) {
 					pstate = ProcessState.Stopped;

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java?rev=1656527&r1=1656526&r2=1656527&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/configuration/jp/HttpWorkerThread.java Mon Feb  2 17:43:11 2015
@@ -266,7 +266,9 @@ public class HttpWorkerThread implements
         					/* *****************************************/
         					logger.warn("run", null,"Terminating Job Process - Work Item Failed");
 
-        					System.exit(0);
+        					// Stop the JVM hard. Agent will check the exit code
+        					// so make it 0 to avoid making it look like CROAK
+        					Runtime.getRuntime().halt(0);
         					/* *****************************************/
         					/* *****************************************/
         					/* *****************************************/