You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hive.apache.org by nz...@apache.org on 2011/10/25 19:07:13 UTC

svn commit: r1188796 - /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java

Author: nzhang
Date: Tue Oct 25 17:07:13 2011
New Revision: 1188796

URL: http://svn.apache.org/viewvc?rev=1188796&view=rev
Log:
HIVE-2446. bug (next one) - If constructor of ClientStatsPublisher throws runtime exception it will be propagated to HadoopJobExecHelper progress method and beyond, whereas it shouldnot

Modified:
    hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java

Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java?rev=1188796&r1=1188795&r2=1188796&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/HadoopJobExecHelper.java Tue Oct 25 17:07:13 2011
@@ -735,8 +735,6 @@ public class HadoopJobExecHelper {
       try {
         clientStatsPublishers.add((ClientStatsPublisher) Class.forName(
             clientStatsPublisherClass.trim(), true, JavaUtils.getClassLoader()).newInstance());
-      } catch (RuntimeException e) {
-        throw e;
       } catch (Exception e) {
         LOG.warn(e.getClass().getName() + " occured when trying to create class: "
             + clientStatsPublisherClass.trim() + " implementing ClientStatsPublisher interface");