You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2018/09/24 07:12:07 UTC

[kylin] branch master updated: KYLIN-3575, fix unclosed JDBC connection in DriverTest.

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 f37be33  KYLIN-3575, fix unclosed JDBC connection in DriverTest.
f37be33 is described below

commit f37be33cedba7fa4f6970d2c3082578e70a6b1a0
Author: tttMelody <24...@qq.com>
AuthorDate: Sun Sep 23 16:53:44 2018 +0800

    KYLIN-3575, fix unclosed JDBC connection in DriverTest.
---
 jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java b/jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java
index 11dbcde..1ffec6d 100644
--- a/jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java
+++ b/jdbc/src/test/java/org/apache/kylin/jdbc/DriverTest.java
@@ -221,6 +221,7 @@ public class DriverTest {
         assertEquals("test_url", ((KylinConnection) conn).getBaseUrl());
         assertEquals("test_db", ((KylinConnection) conn).getProject());
         assertTrue(Boolean.parseBoolean((String) ((KylinConnection) conn).getConnectionProperties().get("ssl")));
+        conn.close();
     }
 
     @Test
@@ -245,6 +246,8 @@ public class DriverTest {
         assertEquals("false", connProps2.getProperty("kylin.query.calcite.extras-props.caseSensitive"));
         assertEquals("UNCHANGED", connProps2.getProperty("kylin.query.calcite.extras-props.unquotedCasing"));
         assertEquals("BACK_TICK", connProps2.getProperty("kylin.query.calcite.extras-props.quoting"));
+        conn.close();
+        conn2.close();
     }
 
     private void printResultSet(ResultSet rs) throws SQLException {