You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by de...@apache.org on 2015/11/12 15:51:03 UTC

svn commit: r1714058 - /uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java

Author: degenaro
Date: Thu Nov 12 14:51:03 2015
New Revision: 1714058

URL: http://svn.apache.org/viewvc?rev=1714058&view=rev
Log:
UIMA-4690 fix AlienFile

Modified:
    uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java

Modified: uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java
URL: http://svn.apache.org/viewvc/uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java?rev=1714058&r1=1714057&r2=1714058&view=diff
==============================================================================
--- uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java (original)
+++ uima/sandbox/uima-ducc/trunk/uima-ducc-common/src/main/java/org/apache/uima/ducc/common/utils/AlienFile.java Thu Nov 12 14:51:03 2015
@@ -71,8 +71,8 @@ public class AlienFile extends AlienAbst
 		String[] command = getCommand();
 		echo(command);
 		ProcessBuilder pb = new ProcessBuilder( command );
+		pb = pb.redirectErrorStream(true);
 		Process p = pb.start();
-		//p.waitFor();
 		InputStream pOut = p.getInputStream();
 		InputStreamReader isr;
 		if(FileHelper.isGzFileType(file_name)) {
@@ -84,12 +84,13 @@ public class AlienFile extends AlienAbst
 		}
 		BufferedReader br = new BufferedReader(isr);
 		int size = 0;
-        int rc = 0;
-        while(rc >= 0) {
-			rc = br.read();
+        int readChar = 0;
+        while(readChar >= 0) {
+			readChar = br.read();
 	        size++;
 		}
-        String text = ""+size;
+        int rc = p.waitFor();
+        String text = "rc="+rc+" "+"size="+size;
         duccLogger.debug(methodName, duccId, text);
         return size;
 	}