Turn off address sanitization by default. We should add a separate bu…

…ild later to test without affecting the runtime anyway.

.github/workflows/build_test.yml

@@ -45,5 +45,5 @@ jobs:
  - name: Build
  run: ninja -C build
  - name: Test
- run: ctest --test-dir build --output-on-failure --timeout 10 --repeat until-pass:2
+ run: ctest --test-dir build --output-on-failure --timeout 30
 

CMakeLists.txt

@@ -14,7 +14,7 @@ set(CMAKE_POSITION_INDEPENDENT_CODE TRUE)
 
 option(SUBSTRAIT_CPP_SANITIZE_DEBUG_BUILD
  "Turns on address and undefined memory sanitization runtime checking."
- ON)
+ OFF)
 
 if(${SUBSTRAIT_CPP_SANITIZE_DEBUG_BUILD})
  add_compile_options($<$<CONFIG:Debug>:-fsanitize=undefined>)

export/planloader/CMakeLists.txt

@@ -1,12 +1,5 @@
 # SPDX-License-Identifier: Apache-2.0
 
-if(CMAKE_BUILD_TYPE MATCHES Debug)
- message(
- WARNING
- "The planloader library does not work well in Debug mode due to bundled heap checking."
- )
-endif()
-
 add_library(planloader SHARED planloader.cpp)
 
 add_dependencies(planloader substrait_io)