Package: clickhouse / 18.16.1+ds-4

Metadata

Package Version Patches format
clickhouse 18.16.1+ds-4 3.0 (quilt)

Patch series

view the series file
Patch File delta Description
0001 Enable link time optimization flto.patch | (download)

CMakeLists.txt | 17 16 + 1 - 0 !
1 file changed, 16 insertions(+), 1 deletion(-)

 enable link time optimization (-flto).

This reduce target's size by 30%.

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0002 Remove non determinism.patch | (download)

dbms/src/Storages/System/StorageSystemBuildOptions.generated.cpp.in | 5 1 + 4 - 0 !
1 file changed, 1 insertion(+), 4 deletions(-)

 remove non-determinism.

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0003 Fix FTBFS with gcc because of Wno format.patch | (download)

contrib/CMakeLists.txt | 8 4 + 4 - 0 !
1 file changed, 4 insertions(+), 4 deletions(-)

 fix ftbfs with gcc because of -wno-format


0004 Disable optimization for build host.patch | (download)

CMakeLists.txt | 2 0 + 2 - 0 !
1 file changed, 2 deletions(-)

 disable optimization for build host.


0005 Use system installed googletest.patch | (download)

cmake/find_gtest.cmake | 8 1 + 7 - 0 !
contrib/CMakeLists.txt | 3 3 + 0 - 0 !
2 files changed, 4 insertions(+), 7 deletions(-)

 use system installed googletest.

Since googletest 3.5 or 3.6 in Debian it is installed as source into
/usr/src/googletest to allow build with user's compiler.

So we just need to build it from the directory the same way we build
bundled one.

0006 Disable unwind.patch | (download)

CMakeLists.txt | 1 0 + 1 - 0 !
1 file changed, 1 deletion(-)

 disable unwind

CH uses unreleaser libunwind API, so disable it until it would be released
and uploaded into the archive.

0007 Disable base64 usage not available in Debian yet.patch | (download)

CMakeLists.txt | 1 0 + 1 - 0 !
1 file changed, 1 deletion(-)

 disable base64 usage (not available in debian yet).


0008 Replace cpuid with cpuinfo.patch | (download)

CMakeLists.txt | 2 1 + 1 - 0 !
cmake/find_cpuid.cmake | 29 0 + 29 - 0 !
cmake/find_cpuinfo.cmake | 16 16 + 0 - 0 !
contrib/CMakeLists.txt | 4 0 + 4 - 0 !
dbms/CMakeLists.txt | 13 8 + 5 - 0 !
dbms/src/Common/config.h.in | 1 1 + 0 - 0 !
dbms/src/Common/getNumberOfPhysicalCPUCores.cpp | 54 17 + 37 - 0 !
7 files changed, 43 insertions(+), 76 deletions(-)

 replace cpuid with cpuinfo.

cpuinfo is already shipped in Debian and provides more suitable API.
This commit is not enough for inclusion into upstream codebase. One also
needs to replace cpuid with cpuinfo in the "contrib" directory.

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0009 Fix install of binaries when SPLIT_BINARIES is set.patch | (download)

dbms/programs/benchmark/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/client/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/compressor/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/copier/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/format/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/odbc-bridge/CMakeLists.txt | 1 1 + 0 - 0 !
dbms/programs/performance-test/CMakeLists.txt | 1 1 + 0 - 0 !
7 files changed, 7 insertions(+)

 fix install of binaries when split_binaries is set.


0010 Fix link with pthread when std thread is used.patch | (download)

dbms/src/Common/Config/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/src/Common/ZooKeeper/tests/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/src/Common/tests/CMakeLists.txt | 8 4 + 4 - 0 !
dbms/src/DataStreams/tests/CMakeLists.txt | 2 1 + 1 - 0 !
4 files changed, 7 insertions(+), 7 deletions(-)

 fix link with pthread when std::thread is used.

According to docs, you should add -lpthread when use std::thread in your code.
This doesn't matter if you build static binaries but fails the build when
one anable SPLIT_SHARED_LIBRARIES.

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0011 Change libraries hierarchy links most of them into m.patch | (download)

cmake/lib_name.cmake | 1 1 + 0 - 0 !
dbms/CMakeLists.txt | 12 8 + 4 - 0 !
dbms/programs/benchmark/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/programs/client/CMakeLists.txt | 4 3 + 1 - 0 !
dbms/programs/copier/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/programs/local/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/programs/odbc-bridge/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/programs/performance-test/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/programs/server/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/src/AggregateFunctions/CMakeLists.txt | 3 1 + 2 - 0 !
dbms/src/Functions/CMakeLists.txt | 8 4 + 4 - 0 !
dbms/src/Storages/System/CMakeLists.txt | 2 1 + 1 - 0 !
dbms/src/TableFunctions/CMakeLists.txt | 4 3 + 1 - 0 !
13 files changed, 27 insertions(+), 19 deletions(-)

 change libraries hierarchy: links most of them into main .so.

I don't thinks this could be applied upstream as is, but allows me to reduce
target's summary size a lot.

0012 Correctly find and use system metrohash library.patch | (download)

cmake/Modules/Findmetrohash.cmake | 2 1 + 1 - 0 !
dbms/src/Functions/CMakeLists.txt | 4 1 + 3 - 0 !
dbms/src/Interpreters/tests/CMakeLists.txt | 2 2 + 0 - 0 !
dbms/src/Interpreters/tests/hash_map_string_3.cpp | 6 3 + 3 - 0 !
4 files changed, 7 insertions(+), 7 deletions(-)

 correctly find and use system metrohash library.

Also fix test conflict with new version of metrahash headers.

I think it would be bundle latest metrohash library into CH's contrib to keep
the codebase synced. (At least it breaks nothing.)

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0013 server config.xml Set log level to warning.patch | (download)

dbms/programs/server/config.xml | 2 1 + 1 - 0 !
1 file changed, 1 insertion(+), 1 deletion(-)

 server/config.xml: set log level to warning.


0014 Set default umask to 027.patch | (download)

libs/libdaemon/src/BaseDaemon.cpp | 5 2 + 3 - 0 !
1 file changed, 2 insertions(+), 3 deletions(-)

 set default umask to 027.

Signed-off-by: Alexander GQ Gerasiov <gq@cs.msu.su>

0015 Fix fallback lz4 decomression.patch | (download)

dbms/src/IO/LZ4_decompress_faster.cpp | 13 4 + 9 - 0 !
1 file changed, 4 insertions(+), 9 deletions(-)

 fix fallback lz4 decomression.