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

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


The following commit(s) were added to refs/heads/master by this push:
     new 910b79e  KYLIN-3603 Close the HBase connection after used in 
UpdateHTableHostCLI
910b79e is described below

commit 910b79e81f415eb9db4fcb414792132ade71df74
Author: Lijun Cao <641507...@qq.com>
AuthorDate: Sat Sep 29 09:11:45 2018 +0800

    KYLIN-3603 Close the HBase connection after used in UpdateHTableHostCLI
---
 .../org/apache/kylin/storage/hbase/util/UpdateHTableHostCLI.java   | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/UpdateHTableHostCLI.java
 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/UpdateHTableHostCLI.java
index bf5c4e8..ae8582c 100644
--- 
a/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/UpdateHTableHostCLI.java
+++ 
b/storage-hbase/src/main/java/org/apache/kylin/storage/hbase/util/UpdateHTableHostCLI.java
@@ -59,9 +59,10 @@ public class UpdateHTableHostCLI {
     public UpdateHTableHostCLI(List<String> htables, String oldHostValue) 
throws IOException {
         this.htables = htables;
         this.oldHostValue = oldHostValue;
-        Connection conn = 
ConnectionFactory.createConnection(HBaseConfiguration.create());
-        hbaseAdmin = conn.getAdmin();
-        this.kylinConfig = KylinConfig.getInstanceFromEnv();
+        try (Connection conn = 
ConnectionFactory.createConnection(HBaseConfiguration.create());) {
+            hbaseAdmin = conn.getAdmin();
+            this.kylinConfig = KylinConfig.getInstanceFromEnv();
+        }
     }
 
     public static void main(String[] args) throws Exception {

Reply via email to