Setting for TCP keepalive. [CLICKHOUSE-1834]

[CurtizJ]

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

[alexey-milovidov]

Let's rename keep_alive_timeout to tcp_keep_alive_timeout everywhere to avoid potential confusion with HTTP Keep-Alive.

[alexey-milovidov]

Missing documentation.

[proller]

this brokes macos build:

FAILED: dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o 
/usr/local/bin/ccache /usr/local/bin/g++-8  -DBOOST_SYSTEM_NO_DEPRECATED -DLZ4_DISABLE_DEPRECATE_WARNINGS=1 -DPOCO_STATIC -DPOCO_UNBUNDLED_ZLIB -DUNALIGNED_OK -DWITH_GZFILEOP -DX86_64 -DZLIB_COMPAT -isystem ../contrib/libsparsehash -isystem ../contrib/libdivide -isystem ../contrib/poco/Data/include -isystem /usr/local/include -I../dbms/src -Idbms/src -isystem ../contrib/double-conversion -isystem ../contrib/libpcg-random/include -I../contrib/cityhash102/include -I../libs/libcommon/include -Ilibs/libcommon/include -I../libs/libpocoext/include -isystem ../contrib/poco/Foundation/include -I../contrib/zlib-ng -Icontrib/zlib-ng -I../contrib/cctz/include -isystem ../contrib/boost -I../libs/libwidechar_width -isystem ../contrib/libcpuid/include -I../contrib/lz4/lib -I../contrib/zstd/lib -isystem ../contrib/poco/Net/include -isystem ../contrib/poco/Util/include -isystem ../contrib/poco/XML/include -isystem ../contrib/poco/JSON/include -isystem ../contrib/poco/NetSSL_OpenSSL/include -isystem ../contrib/poco/Crypto/include -isystem ../contrib/ssl/include -isystem contrib/re2_st -isystem ../contrib/re2 -I../contrib/libbtrie/include -isystem ../contrib/poco/MongoDB/include -I../contrib/capnproto/c++/src -I../contrib/capnproto/c++/src/capnp/.. -I../contrib/capnproto/c++/src/kj/.. -fdiagnostics-color=always  -pipe -msse4.1 -msse4.2 -mpopcnt  -fno-omit-frame-pointer  -Wall  -Wnon-virtual-dtor -Wextra -Werror -O3 -DNDEBUG   -fno-tree-loop-distribute-patterns -std=c++1z -MD -MT dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o -MF dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o.d -o dbms/CMakeFiles/dbms.dir/src/Client/Connection.cpp.o -c ../dbms/src/Client/Connection.cpp
../dbms/src/Client/Connection.cpp: In member function 'void DB::Connection::connect()':
../dbms/src/Client/Connection.cpp:83:31: error: 'SOL_TCP' was not declared in this scope
             socket->setOption(SOL_TCP, TCP_KEEPIDLE, timeouts.tcp_keep_alive_timeout);
                               ^~~~~~~
../dbms/src/Client/Connection.cpp:83:40: error: 'TCP_KEEPIDLE' was not declared in this scope
             socket->setOption(SOL_TCP, TCP_KEEPIDLE, timeouts.tcp_keep_alive_timeout);
                                        ^~~~~~~~~~~~
../dbms/src/Client/Connection.cpp:83:40: note: suggested alternative: 'OCSP_RESPID'
             socket->setOption(SOL_TCP, TCP_KEEPIDLE, timeouts.tcp_keep_alive_timeout);
                                        ^~~~~~~~~~~~
                                        OCSP_RESPID

[jackpgao]

the doc here is still the old.

https://clickhouse.com/docs/en/operations/server-configuration-parameters/settings/#keep-alive-timeout

[morsik]

Apparently this doesn't actually work? :/

#63659