[hbase] branch master updated: HBASE-24844 RecoverableZookeeper#close followed by ExecutorService shutdown

[vj...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 066be4a  HBASE-24844 RecoverableZookeeper#close followed by ExecutorService shutdown
066be4a is described below

commit 066be4a76fdcd825647836d3725f94255759c065
Author: WenFeiYi <we...@gmail.com>
AuthorDate: Wed Aug 12 16:25:49 2020 +0530

    HBASE-24844 RecoverableZookeeper#close followed by ExecutorService shutdown
    
    * Resolves Exception on standalone (master) shutdown
    
    Closes #2239
    
    Signed-off-by: Viraj Jasani <vj...@apache.org>
---
 .../src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java     | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
index 777776f..136134d 100644
--- a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
+++ b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java
@@ -694,11 +694,12 @@ public class ZKWatcher implements Watcher, Abortable, Closeable {
    */
   @Override
   public void close() {
-    zkEventProcessor.shutdownNow();
     try {
       recoverableZooKeeper.close();
     } catch (InterruptedException e) {
       Thread.currentThread().interrupt();
+    } finally {
+      zkEventProcessor.shutdownNow();
     }
   }