Skip to content

Commit cc2cd49

Browse files
committed
remove redundant socket options
1 parent 032f736 commit cc2cd49

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

clickhouse-http-client/src/main/java/com/clickhouse/client/http/ApacheHttpConnectionImpl.java

+5-4
Original file line numberDiff line numberDiff line change
@@ -328,10 +328,11 @@ public HttpConnectionManager(Registry<ConnectionSocketFactory> socketFactory, Cl
328328
.build();
329329
setDefaultConnectionConfig(connConfig);
330330

331-
SocketConfig.Builder builder = SocketConfig.custom()
332-
.setSoTimeout(Timeout.of(config.getSocketTimeout(), TimeUnit.MILLISECONDS))
333-
.setRcvBufSize(config.getReadBufferSize())
334-
.setSndBufSize(config.getWriteBufferSize());
331+
SocketConfig.Builder builder = SocketConfig.custom();
332+
333+
if (config.hasOption(ClickHouseClientOption.SOCKET_TIMEOUT)) {
334+
builder.setSoTimeout(config.getIntOption(ClickHouseClientOption.SOCKET_TIMEOUT), TimeUnit.MILLISECONDS);
335+
}
335336
if (config.hasOption(ClickHouseClientOption.SOCKET_KEEPALIVE)) {
336337
builder.setSoKeepAlive(config.getBoolOption(ClickHouseClientOption.SOCKET_KEEPALIVE));
337338
}

0 commit comments

Comments
 (0)