You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by jd...@apache.org on 2017/05/12 23:40:39 UTC

hive git commit: HIVE-16651: LlapProtocolClientProxy stack trace when using llap input format (Jason Dere, reviewed by Siddharth Seth)

Repository: hive
Updated Branches:
  refs/heads/master 3baaca747 -> fea6df6bc


HIVE-16651: LlapProtocolClientProxy stack trace when using llap input format (Jason Dere, reviewed by Siddharth Seth)


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

Branch: refs/heads/master
Commit: fea6df6bcca6fcd76b95257d838e6d741dcf3f2b
Parents: 3baaca7
Author: Jason Dere <jd...@hortonworks.com>
Authored: Fri May 12 16:39:50 2017 -0700
Committer: Jason Dere <jd...@hortonworks.com>
Committed: Fri May 12 16:39:50 2017 -0700

----------------------------------------------------------------------
 .../test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java    | 1 +
 .../apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java    | 5 ++++-
 2 files changed, 5 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/hive/blob/fea6df6b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
----------------------------------------------------------------------
diff --git a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
index 4cc9045..606a263 100644
--- a/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
+++ b/itests/hive-unit/src/test/java/org/apache/hive/jdbc/TestJdbcWithMiniLlap.java
@@ -425,6 +425,7 @@ public class TestJdbcWithMiniLlap {
         rowProcessor.process(row);
         ++rowCount;
       }
+      reader.close();
     }
 
     return rowCount;

http://git-wip-us.apache.org/repos/asf/hive/blob/fea6df6b/llap-client/src/java/org/apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java
----------------------------------------------------------------------
diff --git a/llap-client/src/java/org/apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java b/llap-client/src/java/org/apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java
index ce75d72..5df19d3 100644
--- a/llap-client/src/java/org/apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java
+++ b/llap-client/src/java/org/apache/hadoop/hive/llap/tez/LlapProtocolClientProxy.java
@@ -24,6 +24,7 @@ import java.util.LinkedList;
 import java.util.List;
 import java.util.Set;
 import java.util.concurrent.Callable;
+import java.util.concurrent.CancellationException;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.ConcurrentMap;
 import java.util.concurrent.ExecutorService;
@@ -129,7 +130,9 @@ public class LlapProtocolClientProxy extends AbstractService {
 
       @Override
       public void onFailure(Throwable t) {
-        LOG.warn("RequestManager shutdown with error", t);
+        if (!(t instanceof CancellationException)) {
+          LOG.warn("RequestManager shutdown with error", t);
+        }
       }
     });
   }