From 4594f7156d98ce4f30c198472830443ee0588275 Mon Sep 17 00:00:00 2001
From: Pankaj Kumar <pankaj.kr@huawei.com>
Date: Tue, 13 Feb 2018 08:43:43 -0800
Subject: [PATCH] HBASE-19979 ReplicationSyncUp tool may leak Zookeeper
 connection

---
 .../hbase/replication/regionserver/ReplicationSyncUp.java   | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSyncUp.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSyncUp.java
index a5cd9b22594..43c7bcee6e1 100644
--- a/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSyncUp.java
+++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/replication/regionserver/ReplicationSyncUp.java
@@ -114,14 +114,14 @@ public class ReplicationSyncUp extends Configured implements Tool {
         Thread.sleep(SLEEP_TIME);
         numberOfOldSource = manager.getOldSources().size();
       }
+      manager.join();
     } catch (InterruptedException e) {
       System.err.println("didn't wait long enough:" + e);
       return (-1);
+    } finally {
+      zkw.close();
     }
 
-    manager.join();
-    zkw.close();
-
     return (0);
   }