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 <245915...@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 {

Reply via email to