You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hive.apache.org by ab...@apache.org on 2020/04/07 16:59:07 UTC

[hive] branch master updated: HIVE-23144: LLAP: Let QueryTracker cleanup on serviceStop (László Bodor reviewed by Prasanth Jayachandran)

This is an automated email from the ASF dual-hosted git repository.

abstractdog pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hive.git


The following commit(s) were added to refs/heads/master by this push:
     new b815167  HIVE-23144: LLAP: Let QueryTracker cleanup on serviceStop (László Bodor reviewed by Prasanth Jayachandran)
b815167 is described below

commit b81516789acc778afca8a4543ae0d7f6533a2a3f
Author: László Bodor <bo...@gmail.com>
AuthorDate: Tue Apr 7 18:58:33 2020 +0200

    HIVE-23144: LLAP: Let QueryTracker cleanup on serviceStop (László Bodor reviewed by Prasanth Jayachandran)
    
    Signed-off-by: Laszlo Bodor <bo...@gmail.com>
---
 .../java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java b/llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java
index 0d5713a..eae8e08 100644
--- a/llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java
+++ b/llap-server/src/java/org/apache/hadoop/hive/llap/daemon/impl/QueryTracker.java
@@ -437,7 +437,12 @@ public class QueryTracker extends AbstractService {
 
   @Override
   public void serviceStop() {
-    executorService.shutdownNow();
+    executorService.shutdown();
+    try {
+      executorService.awaitTermination(10000, TimeUnit.MILLISECONDS);
+    } catch (InterruptedException e) {
+      LOG.warn("cannot finish QueryTracker cleanup because of InterruptedException", e);
+    }
     LOG.info(getName() + " stopped");
   }