Fix split shared build; fix build without system odbc libs (#2191)

* Fix split shared build * Fix build without system odbc libs * Move dbms/src/Common/iostream_debug_helpers.h -> libs/libcommon/include/common/iostream_debug_helpers.h * Fix build with libiodbc

Fix split shared build; fix build without system odbc libs (#2191)
* Fix split shared build * Fix build without system odbc libs * Move dbms/src/Common/iostream_debug_helpers.h -> libs/libcommon/include/common/iostream_debug_helpers.h * Fix build with libiodbc
c26b3bde · proller · GitHub · f02239a3 · c26b3bde · c26b3bde
23 changed file
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -252,6 +252,7 @@ if (EXISTS ${CMAKE_CURRENT_SOURCE_DIR}/contrib/poco/cmake/FindODBC.cmake)
 else ()
    include (cmake/find_odbc.cmake)
 endif ()
+message (STATUS "Using odbc: ${ODBC_INCLUDE_DIRECTORIES} : ${ODBC_LIBRARIES}")
 include (cmake/find_poco.cmake)
 include (cmake/find_lz4.cmake)
 include (cmake/find_sparsehash.cmake)

--- a/cmake/Modules/FindPoco.cmake
+++ b/cmake/Modules/FindPoco.cmake
@@ -51,6 +51,7 @@

 set(Poco_HINTS
    /usr/local
+    /usr/local/include/Poco
    C:/AppliedInformatics
    ${Poco_DIR}
    $ENV{Poco_DIR}
@@ -230,5 +231,3 @@ if(${Poco_OSP_FOUND})
 endif()

 message(STATUS "Found Poco: ${Poco_LIBRARIES}")
-
-
--- a/cmake/find_poco.cmake
+++ b/cmake/find_poco.cmake
@@ -52,26 +52,33 @@ elseif (NOT MISSING_INTERNAL_POCO_LIBRARY)
        set (Poco_MongoDB_INCLUDE_DIRS "${ClickHouse_SOURCE_DIR}/contrib/poco/MongoDB/include/")
    endif ()

-    if (ODBC_FOUND)
-        if (EXISTS "${ClickHouse_SOURCE_DIR}/contrib/poco/SQL/ODBC/include/")
-            set (Poco_SQL_FOUND 1)
-            set (Poco_SQL_LIBRARY PocoSQL)
-            set (Poco_SQL_INCLUDE_DIRS
-                "${ClickHouse_SOURCE_DIR}/contrib/poco/SQL/include"
-                "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/include"
-                )
+
+    if (EXISTS "${ClickHouse_SOURCE_DIR}/contrib/poco/SQL/ODBC/include/")
+        set (Poco_SQL_FOUND 1)
+        set (Poco_SQL_LIBRARY PocoSQL)
+        set (Poco_SQL_INCLUDE_DIRS
+             "${ClickHouse_SOURCE_DIR}/contrib/poco/SQL/include"
+             "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/include"
+             )
+        if (ODBC_FOUND)
            set (Poco_SQLODBC_FOUND 1)
            set (Poco_SQLODBC_INCLUDE_DIRS
                "${ClickHouse_SOURCE_DIR}/contrib/poco/SQL/ODBC/include/"
                "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/ODBC/include/"
+                ${ODBC_INCLUDE_DIRECTORIES}
                )
            set (Poco_SQLODBC_LIBRARY PocoSQLODBC ${ODBC_LIBRARIES} ${LTDL_LIBRARY})
-        else ()
-            set (Poco_Data_FOUND 1)
-            set (Poco_Data_INCLUDE_DIRS "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/include")
-            set (Poco_Data_LIBRARY PocoData)
+        endif ()
+    else ()
+        set (Poco_Data_FOUND 1)
+        set (Poco_Data_INCLUDE_DIRS "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/include")
+        set (Poco_Data_LIBRARY PocoData)
+        if (ODBC_FOUND)
            set (Poco_DataODBC_FOUND 1)
-            set (Poco_DataODBC_INCLUDE_DIRS "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/ODBC/include/")
+            set (Poco_DataODBC_INCLUDE_DIRS
+                "${ClickHouse_SOURCE_DIR}/contrib/poco/Data/ODBC/include/"
+                ${ODBC_INCLUDE_DIRECTORIES}
+            )
            set (Poco_DataODBC_LIBRARY PocoDataODBC ${ODBC_LIBRARIES} ${LTDL_LIBRARY})
        endif ()
    endif ()

--- a/dbms/src/Common/Exception.cpp
+++ b/dbms/src/Common/Exception.cpp
@@ -11,7 +11,7 @@
 #include <IO/ReadBufferFromString.h>

 #include <Common/Exception.h>
-#include <Common/demangle.h>
+#include <common/demangle.h>


 namespace DB

--- a/dbms/src/Common/FieldVisitors.h
+++ b/dbms/src/Common/FieldVisitors.h
@@ -3,7 +3,7 @@
 #include <Core/Field.h>
 #include <Core/AccurateComparison.h>
 #include <common/DateLUT.h>
-#include <Common/demangle.h>
+#include <common/demangle.h>


 class SipHash;

--- a/dbms/src/Common/StackTrace.cpp
+++ b/dbms/src/Common/StackTrace.cpp
@@ -7,7 +7,7 @@
 #include <sstream>

 #include <Common/StackTrace.h>
-#include <Common/demangle.h>
+#include <common/demangle.h>


 StackTrace::StackTrace()

--- a/dbms/src/Common/Visitor.h
+++ b/dbms/src/Common/Visitor.h
 #pragma once

-#include <Common/demangle.h>
+#include <common/demangle.h>
 #include <Common/TypeList.h>
 #include <Common/Exception.h>


--- a/dbms/src/Common/tests/CMakeLists.txt
+++ b/dbms/src/Common/tests/CMakeLists.txt
@@ -71,6 +71,3 @@ target_link_libraries (cow_columns clickhouse_common_io)

 add_executable (stopwatch stopwatch.cpp)
 target_link_libraries (stopwatch clickhouse_common_io)
-
-add_executable (dump_variable dump_variable.cpp)
-target_link_libraries (dump_variable clickhouse_common_io)
--- a/dbms/src/Common/typeid_cast.h
+++ b/dbms/src/Common/typeid_cast.h
@@ -6,7 +6,7 @@
 #include <string>

 #include <Common/Exception.h>
-#include <Common/demangle.h>
+#include <common/demangle.h>


 namespace DB

--- a/dbms/src/Core/iostream_debug_helpers.h
+++ b/dbms/src/Core/iostream_debug_helpers.h
@@ -45,4 +45,4 @@ std::ostream & operator<<(std::ostream & stream, const Connection::Packet & what
 }

 /// some operator<< should be declared before operator<<(... std::shared_ptr<>)
-#include <Common/iostream_debug_helpers.h>
+#include <common/iostream_debug_helpers.h>
--- a/dbms/src/Functions/FunctionsMiscellaneous.cpp
+++ b/dbms/src/Functions/FunctionsMiscellaneous.cpp
@@ -31,7 +31,7 @@
 #include <Interpreters/ExpressionActions.h>
 #include <Storages/IStorage.h>
 #include <Common/typeid_cast.h>
-#include <TableFunctions/getStructureOfRemoteTable.h>
+#include <Storages/getStructureOfRemoteTable.h>


 namespace DB

--- a/dbms/src/Interpreters/Aggregator.cpp
+++ b/dbms/src/Interpreters/Aggregator.cpp
@@ -22,7 +22,7 @@
 #include <Common/ClickHouseRevision.h>
 #include <Common/MemoryTracker.h>
 #include <Common/typeid_cast.h>
-#include <Common/demangle.h>
+#include <common/demangle.h>
 #include <Interpreters/config_compile.h>



--- a/dbms/src/TableFunctions/getStructureOfRemoteTable.cpp
+++ b/dbms/src/TableFunctions/getStructureOfRemoteTable.cpp
-#include <TableFunctions/getStructureOfRemoteTable.h>
+#include "getStructureOfRemoteTable.h"
 #include <Interpreters/Cluster.h>
 #include <Interpreters/Context.h>
 #include <Interpreters/InterpreterDescribeQuery.h>

--- a/dbms/src/TableFunctions/getStructureOfRemoteTable.h
+++ b/dbms/src/TableFunctions/getStructureOfRemoteTable.h
--- a/dbms/src/TableFunctions/CMakeLists.txt
+++ b/dbms/src/TableFunctions/CMakeLists.txt
@@ -5,7 +5,7 @@ list(REMOVE_ITEM clickhouse_table_functions_sources ITableFunction.cpp TableFunc
 list(REMOVE_ITEM clickhouse_table_functions_headers ITableFunction.h TableFunctionFactory.h)

 add_library(clickhouse_table_functions ${clickhouse_table_functions_sources})
-target_link_libraries(clickhouse_table_functions dbms ${Poco_Foundation_LIBRARY})
+target_link_libraries(clickhouse_table_functions clickhouse_storages_system dbms ${Poco_Foundation_LIBRARY})

 if (Poco_SQLODBC_FOUND)
    target_link_libraries (clickhouse_table_functions ${Poco_SQLODBC_LIBRARY})

--- a/dbms/src/TableFunctions/TableFunctionRemote.cpp
+++ b/dbms/src/TableFunctions/TableFunctionRemote.cpp
-#include <TableFunctions/getStructureOfRemoteTable.h>
+#include <Storages/getStructureOfRemoteTable.h>
 #include <Storages/StorageDistributed.h>
 #include <Parsers/ASTIdentifier.h>
 #include <Parsers/ASTLiteral.h>

--- a/dbms/src/TableFunctions/TableFunctionShardByHash.cpp
+++ b/dbms/src/TableFunctions/TableFunctionShardByHash.cpp
-#include <TableFunctions/getStructureOfRemoteTable.h>
+#include <Storages/getStructureOfRemoteTable.h>
 #include <Storages/StorageDistributed.h>
 #include <Parsers/ASTIdentifier.h>
 #include <Parsers/ASTLiteral.h>

--- a/libs/libcommon/CMakeLists.txt
+++ b/libs/libcommon/CMakeLists.txt
@@ -26,6 +26,7 @@ add_library (common ${SPLIT_SHARED}
    src/JSON.cpp
    src/getMemoryAmount.cpp
    src/ThreadPool.cpp
+    src/demangle.cpp

    include/common/Types.h
    include/common/DateLUT.h
@@ -44,6 +45,7 @@ add_library (common ${SPLIT_SHARED}
    include/common/JSON.h
    include/common/getMemoryAmount.h
    include/common/ThreadPool.h
+    include/common/demangle.h

    include/ext/bit_cast.h
    include/ext/collection_cast.h

--- a/dbms/src/Common/demangle.h
+++ b/dbms/src/Common/demangle.h
--- a/dbms/src/Common/iostream_debug_helpers.h
+++ b/dbms/src/Common/iostream_debug_helpers.h
 #pragma once

-#include <Common/demangle.h>
+#include <common/demangle.h>

 #include <type_traits>
 #include <tuple>

--- a/dbms/src/Common/demangle.cpp
+++ b/dbms/src/Common/demangle.cpp
-#include <Common/demangle.h>
+#include <common/demangle.h>
 #include <cxxabi.h>
 #include <stdlib.h>


--- a/libs/libcommon/src/tests/CMakeLists.txt
+++ b/libs/libcommon/src/tests/CMakeLists.txt
@@ -20,3 +20,6 @@ add_check(multi_version)
 add_executable (unit_tests_libcommon gtest_json_test.cpp gtest_strong_typedef.cpp)
 target_link_libraries (unit_tests_libcommon gtest_main common)
 add_check(unit_tests_libcommon)
+
+add_executable (dump_variable dump_variable.cpp)
+target_link_libraries (dump_variable clickhouse_common_io)
--- a/dbms/src/Common/tests/dump_variable.cpp
+++ b/dbms/src/Common/tests/dump_variable.cpp
-#include <Common/iostream_debug_helpers.h>
+#include <common/iostream_debug_helpers.h>

 #include <iostream>
 #include <memory>