Skip to content

Commit 37880d4

Browse files
committed
removed unused import and potential NPE
1 parent d34c6e1 commit 37880d4

2 files changed

Lines changed: 3 additions & 4 deletions

File tree

jdbc-v2/src/main/java/com/clickhouse/jdbc/ConnectionImpl.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
import com.clickhouse.client.api.Client;
44
import com.clickhouse.client.api.ClientConfigProperties;
5-
import com.clickhouse.client.api.internal.ServerSettings;
65
import com.clickhouse.client.api.metadata.TableSchema;
76
import com.clickhouse.client.api.query.GenericRecord;
87
import com.clickhouse.client.api.query.QuerySettings;

jdbc-v2/src/test/java/com/clickhouse/jdbc/JdbcIntegrationTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,13 @@
55
import com.clickhouse.client.ClickHouseServerForTest;
66
import com.clickhouse.client.api.ClientConfigProperties;
77
import com.clickhouse.client.api.internal.ServerSettings;
8-
import com.clickhouse.client.api.query.GenericRecord;
98
import com.clickhouse.data.ClickHouseVersion;
109
import com.clickhouse.logging.Logger;
1110
import com.clickhouse.logging.LoggerFactory;
1211

1312
import java.sql.Connection;
1413
import java.sql.SQLException;
1514
import java.sql.Statement;
16-
import java.util.List;
1715
import java.util.Map;
1816
import java.util.Properties;
1917

@@ -39,7 +37,9 @@ public Connection getJdbcConnection() throws SQLException {
3937

4038
public Connection getJdbcConnection(Map<String, Object> propertiesMap) throws SQLException {
4139
Properties config = new Properties();
42-
config.putAll(propertiesMap);
40+
if (propertiesMap != null && !propertiesMap.isEmpty()) {
41+
config.putAll(propertiesMap);
42+
}
4343
return getJdbcConnection(config);
4444
}
4545

0 commit comments

Comments
 (0)