Copilot commented on code in PR #49668:
URL: https://github.com/apache/arrow/pull/49668#discussion_r3270735284
##########
ci/scripts/cpp_test.sh:
##########
@@ -55,6 +55,7 @@ if ! type minio >/dev/null 2>&1; then
fi
case "$(uname)" in
Linux)
+ exclude_tests+=("arrow-flight-sql-odbc-test")
n_jobs=$(nproc)
;;
Review Comment:
On Linux, this change unconditionally adds `arrow-flight-sql-odbc-test` to
the ctest exclude list, so the ODBC test binary will be built but never
executed in Linux CI. If the intent is only “build-only” support, consider
adding an explicit TODO/issue reference here; otherwise, gate the exclusion on
missing runtime prerequisites (e.g., missing driver manager config) or add a
dedicated Linux CI job that runs the test so the new Linux build support is
actually exercised.
--
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]
