Skip to content

Commit d9c3ff7

Browse files
committed
merge with master
2 parents c6b2ed8 + 35bc659 commit d9c3ff7

File tree

5 files changed

+6
-27
lines changed

5 files changed

+6
-27
lines changed

CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -665,10 +665,6 @@ list(APPEND WARPDRIVE_SHARED_INSTALL_INTERFACE_LIBS odbcabstraction)
665665

666666
# Add include directories for _only_ the ODBCAbstraction SPI.
667667
include_directories(${CMAKE_CURRENT_BINARY_DIR}/ODBCAbstraction-prefix/src/ODBCAbstraction/odbcabstraction/include)
668-
include_directories(${CMAKE_CURRENT_BINARY_DIR}/ODBCAbstraction-prefix/src/ODBCAbstraction-build/_deps/spdlog-src/include)
669-
670-
# Ensure fmt is loaded as header only
671-
add_compile_definitions(FMT_HEADER_ONLY)
672668

673669
#
674670
# Linker and Dependencies

src/arrow-odbc/CMakeLists.txt

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,6 @@ find_package(OpenSSL REQUIRED)
3434
# Add gRPC dependencies needed by Arrow Flight. Should be pre-installed.
3535
find_package(gRPC 1.36 CONFIG REQUIRED)
3636

37-
# Ensure fmt is loaded as header only
38-
add_compile_definitions(FMT_HEADER_ONLY)
39-
4037
# Define the version number information.
4138
if(DEFINED ENV{WARPDRIVE_BUILD_VERSION})
4239
set(WARPDRIVE_BUILD_VERSION "$ENV{WARPDRIVE_BUILD_VERSION}")
@@ -156,10 +153,10 @@ include_directories(arrow-odbc
156153
${CMAKE_CURRENT_BINARY_DIR}/../warpdrive
157154
${CMAKE_CURRENT_BINARY_DIR}/../../ODBCAbstraction-prefix/src/ODBCAbstraction/flight_sql/include
158155
${CMAKE_CURRENT_BINARY_DIR}/../../ODBCAbstraction-prefix/src/ODBCAbstraction/odbcabstraction/include
159-
${CMAKE_CURRENT_BINARY_DIR}/../../ODBCAbstraction-prefix/src/ODBCAbstraction-build/_deps/spdlog-src/include
160156
${ODBC_INCLUDE_DIRS}
161157
${Boost_INCLUDE_DIRS}
162158
)
163159

164160
link_directories(${BUILD_OUTPUT_ROOT_DIRECTORY})
165-
configure_file(arrow-odbc.ini.in ${BUILD_OUTPUT_ROOT_DIRECTORY}/arrow-odbc.ini @ONLY)
161+
162+
#target_link_libraries(FirstTest gtest gtest_main pthread)

src/arrow-odbc/arrow-odbc.ini.in

Lines changed: 0 additions & 5 deletions
This file was deleted.

src/warpdrive/CMakeLists.txt

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,6 @@ add_dependencies(warpdrive-all
1414
# Enable unicode driver build
1515
add_compile_definitions(UNICODE_SUPPORT)
1616

17-
# Ensure fmt is loaded as header only
18-
add_compile_definitions(FMT_HEADER_ONLY)
19-
2017
include_directories(${ODBC_INCLUDE_DIRS})
2118

2219

@@ -144,7 +141,6 @@ add_dependencies(warpdrive ${WARPDRIVE_LIBRARIES})
144141
add_dependencies(warpdrive ODBCAbstraction)
145142
include_directories(warpdrive
146143
${CMAKE_CURRENT_BINARY_DIR}/../../ODBCAbstraction-prefix/src/ODBCAbstraction/odbcabstraction/include
147-
${CMAKE_CURRENT_BINARY_DIR}/../../ODBCAbstraction-prefix/src/ODBCAbstraction-build/_deps/spdlog-src/include
148144
${Boost_INCLUDE_DIRS}
149145
)
150146

src/warpdrive/environ.cc

Lines changed: 4 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@
3131
#include "loadlib.h"
3232
#include "new_driver.h"
3333
#include <odbcabstraction/odbc_impl/ODBCEnvironment.h>
34-
#include <odbcabstraction/spi/driver.h>
35-
#include <odbcabstraction/logger.h>
3634

3735

3836
/* The one instance of the handles */
@@ -73,13 +71,10 @@ namespace {
7371
static std::mutex s_driverLock;
7472

7573
void InitializeDriverIfNeeded() {
76-
std::lock_guard<std::mutex> lock(s_driverLock);
77-
if (s_driver == nullptr) {
78-
s_driver = CreateDriver();
79-
s_driver->RegisterLog();
80-
81-
LOG_DEBUG("Driver initialized");
82-
}
74+
std::lock_guard<std::mutex> lock(s_driverLock);
75+
if (s_driver == nullptr) {
76+
s_driver = CreateDriver();
77+
}
8378
}
8479
}
8580

0 commit comments

Comments
 (0)