You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by th...@apache.org on 2013/09/19 04:57:56 UTC

svn commit: r1524618 - /hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java

Author: thejas
Date: Thu Sep 19 02:57:56 2013
New Revision: 1524618

URL: http://svn.apache.org/r1524618
Log:
HIVE-5198: WebHCat returns exitcode 143 (w/o an explanation) (Eugene Koifman via Thejas Nair)

Modified:
    hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java

Modified: hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java
URL: http://svn.apache.org/viewvc/hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java?rev=1524618&r1=1524617&r2=1524618&view=diff
==============================================================================
--- hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java (original)
+++ hive/branches/branch-0.12/hcatalog/webhcat/svr/src/main/java/org/apache/hive/hcatalog/templeton/ExecServiceImpl.java Thu Sep 19 02:57:56 2013
@@ -137,6 +137,18 @@ public class ExecServiceImpl implements 
     String enc = appConf.get(AppConfig.EXEC_ENCODING_NAME);
     res.stdout = outStream.toString(enc);
     res.stderr = errStream.toString(enc);
+    try {
+      watchdog.checkException();
+    }
+    catch (Exception ex) {
+      LOG.error("Command: " + cmd + " failed:", ex);
+    }
+    if(watchdog.killedProcess()) {
+      String msg = " was terminated due to timeout(" + timeout + "ms).  See " + AppConfig
+              .EXEC_TIMEOUT_NAME + " property"; 
+      LOG.warn("Command: " + cmd + msg);
+      res.stderr += " Command " + msg; 
+    }
 
     return res;
   }