Author: burn
Date: Wed Jul  9 22:31:49 2014
New Revision: 1609333

URL: http://svn.apache.org/r1609333
Log:
UIMA-3933 Set DUCC_PROCESS_LOG_PREFIX in environment for APs & jobs

Modified:
    
uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/DuccCommandExecutor.java

Modified: 
uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/DuccCommandExecutor.java
URL: 
http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/DuccCommandExecutor.java?rev=1609333&r1=1609332&r2=1609333&view=diff
==============================================================================
--- 
uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/DuccCommandExecutor.java
 (original)
+++ 
uima/sandbox/uima-ducc/trunk/uima-ducc-agent/src/main/java/org/apache/uima/ducc/agent/launcher/DuccCommandExecutor.java
 Wed Jul  9 22:31:49 2014
@@ -53,7 +53,6 @@ public class DuccCommandExecutor extends
                        NodeAgent.COMPONENT_NAME);
        @SuppressWarnings("unused")
        private static AtomicInteger nextPort = new AtomicInteger(30000);
-       private static int SIGTERM_KILL_EXITCODE = 143;
        
        public DuccCommandExecutor(NodeAgent agent, ICommandLine cmdLine,String 
host, String ip, Process managedProcess)
                        throws Exception {
@@ -470,7 +469,7 @@ public class DuccCommandExecutor extends
              switch( 
((ManagedProcess)super.managedProcess).getDuccProcess().getProcessType() ) {
                case Pop:
                  // Both JD and POP arbitrary process are POPs. Assume this is 
an arbitrary process
-            processType = "-POP-";  
+                 processType = "-POP-";  
                  if ( cmdLine instanceof JavaCommandLine ) {
                    List<String> options = 
((JavaCommandLine)cmdLine).getOptions();
                    for(String option : options ) {
@@ -485,13 +484,12 @@ public class DuccCommandExecutor extends
                  }
                  break;
                case Service:
-            //processType = "-AP-";
+              //processType = "-AP-";
                  break;
                case Job_Uima_AS_Process:
-            processType = "-UIMA-";
-            
((JavaCommandLine)cmdLine).addOption("-Dducc.deploy.components=uima-as");
-            
((JavaCommandLine)cmdLine).setClassName("org.apache.uima.ducc.common.main.DuccService");
-            
+              processType = "-UIMA-";
+              
((JavaCommandLine)cmdLine).addOption("-Dducc.deploy.components=uima-as");
+              
((JavaCommandLine)cmdLine).setClassName("org.apache.uima.ducc.common.main.DuccService");
                  break;
              }
 //           if ( 
((ManagedProcess)super.managedProcess).getDuccProcess().getProcessType().equals(ProcessType.Pop))
 {
@@ -532,6 +530,7 @@ public class DuccCommandExecutor extends
                    
operationalProperties.add("-D"+NodeAgent.ProcessStateUpdatePort+"="+System.getProperty(NodeAgent.ProcessStateUpdatePort));
                  }
                }
+               // NOTE - These are redundant since the information is also in 
the environment for both Java and non-Java processes
                
operationalProperties.add("-Dducc.process.log.dir="+processLogDir);
                
operationalProperties.add("-Dducc.process.log.basename="+processLogFile); 
//((ManagedProcess)super.managedProcess).getWorkDuccId()+ processType+host);
                
operationalProperties.add("-Dducc.job.id="+((ManagedProcess)super.managedProcess).getWorkDuccId());
@@ -544,10 +543,9 @@ public class DuccCommandExecutor extends
              } else {
                cmd = Utils.concatAllArrays(new String[] {executable}, 
operationalProperties.toArray(operationalPropertiesArray), 
cmdLine.getCommandLine());
              }
-             // add JobId to the env
-             if ( processEnv != null ) {
-               processEnv.put("JobId", 
String.valueOf(((ManagedProcess)super.managedProcess).getWorkDuccId().getFriendly()));
-             }
+             // add JobId and the log prefix to the env so additional 
similarly-named log files can be created
+             processEnv.put("JobId", 
String.valueOf(((ManagedProcess)super.managedProcess).getWorkDuccId().getFriendly()));
+             processEnv.put("DUCC_PROCESS_LOG_PREFIX", processLogDir + 
processLogFile);
            }
            return cmd;
          } catch( Exception ex) {


Reply via email to