This is an automated email from the ASF dual-hosted git repository.

raulcd pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new 8febeadf22 GH-46052: [C++][Benchmarking] Don't build grouper benchmark 
without ARROW_COMPUTE=ON (#46053)
8febeadf22 is described below

commit 8febeadf225b1019d561f91191200faa16afd072
Author: Sutou Kouhei <[email protected]>
AuthorDate: Tue Apr 8 16:44:44 2025 +0900

    GH-46052: [C++][Benchmarking] Don't build grouper benchmark without 
ARROW_COMPUTE=ON (#46053)
    
    ### Rationale for this change
    
    We should not build benchmark program that uses compute module features 
without `ARROW_COMPUTE=ON`.
    
    ### What changes are included in this PR?
    
    Don't build `grouper_benchmark.cc` without `ARROW_COMPUTE=ON`.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    Yes.
    * GitHub Issue: #46052
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Raúl Cumplido <[email protected]>
---
 cpp/src/arrow/compute/row/CMakeLists.txt | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/cpp/src/arrow/compute/row/CMakeLists.txt 
b/cpp/src/arrow/compute/row/CMakeLists.txt
index ef03c767f9..747fd0a92d 100644
--- a/cpp/src/arrow/compute/row/CMakeLists.txt
+++ b/cpp/src/arrow/compute/row/CMakeLists.txt
@@ -20,4 +20,6 @@
 
 arrow_install_all_headers("arrow/compute/row")
 
-add_arrow_benchmark(grouper_benchmark PREFIX "arrow-compute")
+if(ARROW_COMPUTE)
+  add_arrow_benchmark(grouper_benchmark PREFIX "arrow-compute")
+endif()

Reply via email to