Skip to content

Commit 51590b5

Browse files
authored
Merge pull request #1263 from JackyWoo/remove_redundant_socket_options
remove redundant socket options
2 parents 2651291 + a8a5db9 commit 51590b5

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

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

-8
Original file line numberDiff line numberDiff line change
@@ -342,14 +342,6 @@ public HttpConnectionManager(Registry<ConnectionSocketFactory> socketFactory, Cl
342342
if (config.hasOption(ClickHouseClientOption.SOCKET_REUSEADDR)) {
343343
builder.setSoReuseAddress(config.getBoolOption(ClickHouseClientOption.SOCKET_REUSEADDR));
344344
}
345-
if (config.hasOption(ClickHouseClientOption.SOCKET_RCVBUF)) {
346-
int bufferSize = config.getIntOption(ClickHouseClientOption.SOCKET_RCVBUF);
347-
builder.setRcvBufSize(bufferSize > 0 ? bufferSize : config.getReadBufferSize());
348-
}
349-
if (config.hasOption(ClickHouseClientOption.SOCKET_SNDBUF)) {
350-
int bufferSize = config.getIntOption(ClickHouseClientOption.SOCKET_SNDBUF);
351-
builder.setSndBufSize(bufferSize > 0 ? bufferSize : config.getWriteBufferSize());
352-
}
353345
if (config.hasOption(ClickHouseClientOption.SOCKET_TCP_NODELAY)) {
354346
builder.setTcpNoDelay(config.getBoolOption(ClickHouseClientOption.SOCKET_TCP_NODELAY));
355347
}

0 commit comments

Comments
 (0)