alinaliBQ commented on code in PR #49784:
URL: https://github.com/apache/arrow/pull/49784#discussion_r3113561427
##########
cpp/src/arrow/flight/sql/odbc/tests/CMakeLists.txt:
##########
@@ -42,6 +42,11 @@ set(ARROW_FLIGHT_SQL_ODBC_TEST_SRCS
# GH-46889: move protobuf_test_util to a more common location
../../../../engine/substrait/protobuf_test_util.cc)
+# Resolve segmentation fault error on Windows platform due to Arrow Compute
Library Initialization
+if(WIN32)
+ list(APPEND ARROW_FLIGHT_SQL_ODBC_TEST_SRCS ../../../../compute/test_env.cc)
+endif()
Review Comment:
@lidavidm Is it ok if we only apply
`ASSERT_OK(arrow::compute::Initialize());` to Windows (where the crash occurs)?
On macOS and Linux, it would be better to test the ODBC driver's behavior of
compute call initialization in the tests where possible.
cc @justing-bq
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
