You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by br...@apache.org on 2014/08/13 19:22:11 UTC
svn commit: r1617771 -
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java
Author: brock
Date: Wed Aug 13 17:22:11 2014
New Revision: 1617771
URL: http://svn.apache.org/r1617771
Log:
HIVE-7697 - PlanUtils.getTableDesc uses printStackTrace and returns null (Brock reviewed by Navis)
Modified:
hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java
Modified: hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java
URL: http://svn.apache.org/viewvc/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java?rev=1617771&r1=1617770&r2=1617771&view=diff
==============================================================================
--- hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java (original)
+++ hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/PlanUtils.java Wed Aug 13 17:22:11 2014
@@ -157,7 +157,7 @@ public final class PlanUtils {
} catch (ClassNotFoundException e) {
// mimicking behaviour in CreateTableDesc tableDesc creation
// returning null table description for output.
- e.printStackTrace();
+ LOG.warn("Unable to find class in getDefaultTableDesc: " + e.getMessage(), e);
return null;
}
return ret;
@@ -364,8 +364,7 @@ public final class PlanUtils {
ret.setInputFileFormatClass(in_class);
ret.setOutputFileFormatClass(out_class);
} catch (ClassNotFoundException e) {
- e.printStackTrace();
- return null;
+ throw new RuntimeException("Unable to find class in getTableDesc: " + e.getMessage(), e);
}
return ret;
}