svn commit: r1750411 - in /uima/uima-ducc/trunk: uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java

[degenaro]

Author: degenaro
Date: Mon Jun 27 20:50:49 2016
New Revision: 1750411

URL: http://svn.apache.org/viewvc?rev=1750411&view=rev
Log:
UIMA-4989 Fix problems found by FindBugs

Modified:
    
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java
    
uima/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java

Modified: 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java
URL: 
http://svn.apache.org/viewvc/uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java?rev=1750411&r1=1750410&r2=1750411&view=diff
==============================================================================
--- 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java
 (original)
+++ 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/factory/JobFactory.java
 Mon Jun 27 20:50:49 2016
@@ -627,7 +627,7 @@ public class JobFactory implements IJobF
                        String processEnvironmentVariables = 
jobRequestProperties.getProperty(JobSpecificationProperties.key_environment);
                        int envCountProcess = addEnvironment(job, "process", 
executableProcessCommandLine, processEnvironmentVariables);
                        logger.info(methodName, job.getDuccId(), "process env 
vars: "+envCountProcess);
-                       logger.debug(methodName, job.getDuccId(), "ducclet: 
"+executableProcessCommandLine.getCommandLine());
+                       logger.debug(methodName, job.getDuccId(), "ducclet: 
"+executableProcessCommandLine.getCommandLineString());
                        job.setCommandLine(executableProcessCommandLine);
                        // Tokenize arguments string and strip any quotes, then 
add to command line.
                        // Note: placeholders replaced by CLI so can avoid the 
add method.

Modified: 
uima/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java
URL: 
http://svn.apache.org/viewvc/uima/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java?rev=1750411&r1=1750410&r2=1750411&view=diff
==============================================================================
--- 
uima/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java
 (original)
+++ 
uima/uima-ducc/trunk/uima-ducc-transport/src/main/java/org/apache/uima/ducc/transport/cmdline/NonJavaCommandLine.java
 Mon Jun 27 20:50:49 2016
@@ -30,6 +30,10 @@ public class NonJavaCommandLine extends
                String[] args = new String[super.args.size()];
                return super.args.toArray(args);
        }
+       public String getCommandLineString() {
+               // TODO Auto-generated method stub
+               return null;
+       }
        public static void main(String[] args) {
                try {
                        NonJavaCommandLine cmdLine = new 
NonJavaCommandLine("someExecutable.exe");