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 {