raulcd commented on code in PR #55294:
URL: https://github.com/apache/arrow/pull/55294#discussion_r4409863562
##########
cpp/src/arrow/compute/row/CMakeLists.txt:
##########
@@ -22,7 +22,7 @@ arrow_install_all_headers("arrow/compute/row")
if(ARROW_BUILD_BENCHMARKS AND ARROW_COMPUTE)
add_arrow_benchmark(grouper_benchmark PREFIX "arrow-compute")
- if(ARROW_BUILD_STATIC)
+ if(ARROW_TEST_LINKAGE STREQUAL "static")
Review Comment:
Sorry, my suggestion here was wrong. It seems we are trying to register
protobuf twice which aborts at startup (see the CI failures). I would revert
that change and we can follow up separately if we want ORC tests to support
shared + static.
--
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]
