You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ko...@apache.org on 2023/04/04 23:31:36 UTC

[arrow] branch main updated: MINOR: [C++][Benchmark][Acero] Fix a typo (#34875)

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

kou 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 8824acac00 MINOR: [C++][Benchmark][Acero] Fix a typo (#34875)
8824acac00 is described below

commit 8824acac0064e27409d0c41178c9cf45e6ffc6fc
Author: Sutou Kouhei <ko...@clear-code.com>
AuthorDate: Wed Apr 5 08:31:30 2023 +0900

    MINOR: [C++][Benchmark][Acero] Fix a typo (#34875)
    
    ### Rationale for this change
    
    It causes a build error:
    
    ```text
    cpp/src/arrow/acero/hash_join_benchmark.cc:40:3: error: ‘JoinType’ does not name a type; did you mean ‘UnionType’?
       40 |   JoinType join_type = JoinType::INNER;
          |   ^~~~~~~~
          |   UnionType
    ...
    ```
    
    ### What changes are included in this PR?
    
    Fix a typo.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    No.
---
 cpp/src/arrow/acero/hash_join_benchmark.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpp/src/arrow/acero/hash_join_benchmark.cc b/cpp/src/arrow/acero/hash_join_benchmark.cc
index 7f1653a98c..9be4bed606 100644
--- a/cpp/src/arrow/acero/hash_join_benchmark.cc
+++ b/cpp/src/arrow/acero/hash_join_benchmark.cc
@@ -34,7 +34,7 @@
 #include <omp.h>
 
 namespace arrow {
-namespace aceroacero {
+namespace acero {
 struct BenchmarkSettings {
   int num_threads = 1;
   JoinType join_type = JoinType::INNER;
@@ -453,5 +453,5 @@ BENCHMARK(BM_HashJoinBasic_ProbeParallelism)
 
 #endif  // ARROW_BUILD_DETAILED_BENCHMARKS
 
-}  // namespace aceroacero
+}  // namespace acero
 }  // namespace arrow