You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by pv...@apache.org on 2019/01/21 09:34:18 UTC

hive git commit: HIVE-21099: Do Not Print StackTraces to STDERR in ConditionalResolverMergeFiles (Mani M via Peter Vary)

Repository: hive
Updated Branches:
  refs/heads/master a71297852 -> ccba23f1a


HIVE-21099: Do Not Print StackTraces to STDERR in ConditionalResolverMergeFiles (Mani M via Peter Vary)


Project: http://git-wip-us.apache.org/repos/asf/hive/repo
Commit: http://git-wip-us.apache.org/repos/asf/hive/commit/ccba23f1
Tree: http://git-wip-us.apache.org/repos/asf/hive/tree/ccba23f1
Diff: http://git-wip-us.apache.org/repos/asf/hive/diff/ccba23f1

Branch: refs/heads/master
Commit: ccba23f1a44217c90df0038e35a3ee50f2290a9a
Parents: a712978
Author: Mani M <rm...@gmail.com>
Authored: Fri Jan 18 15:50:02 2019 +0100
Committer: Peter Vary <pv...@cloudera.com>
Committed: Mon Jan 21 10:24:26 2019 +0100

----------------------------------------------------------------------
 .../hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java     | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/ccba23f1/ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java
----------------------------------------------------------------------
diff --git a/ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java b/ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java
index 80f77b9..54c9659 100644
--- a/ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java
+++ b/ql/src/java/org/apache/hadoop/hive/ql/plan/ConditionalResolverMergeFiles.java
@@ -34,6 +34,8 @@ import org.apache.hadoop.hive.conf.HiveConf;
 import org.apache.hadoop.hive.metastore.Warehouse;
 import org.apache.hadoop.hive.ql.exec.Task;
 import org.apache.hadoop.hive.ql.exec.Utilities;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /**
  * Conditional task resolution interface. This is invoked at run time to get the
@@ -43,6 +45,8 @@ public class ConditionalResolverMergeFiles implements ConditionalResolver,
     Serializable {
   private static final long serialVersionUID = 1L;
 
+  private static final Logger LOG = LoggerFactory.getLogger(ConditionalResolverMergeFiles.class);
+
   public ConditionalResolverMergeFiles() {
   }
 
@@ -190,7 +194,7 @@ public class ConditionalResolverMergeFiles implements ConditionalResolver,
         resTsks.add(mvTask);
       }
     } catch (IOException e) {
-      e.printStackTrace();
+      LOG.warn("Exception while getting tasks", e);
     }
 
     // Only one of the tasks should ever be added to resTsks