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

apitrou 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 40a8a68c7c GH-40233: [C++] Fix an abort on asof_join_benchmark run for 
lost an arg (#40234)
40a8a68c7c is described below

commit 40a8a68c7c3903e2bad98605d82339460c5ea930
Author: ZhangHuiGui <[email protected]>
AuthorDate: Tue Feb 27 22:00:59 2024 +0800

    GH-40233: [C++] Fix an abort on asof_join_benchmark run for lost an arg 
(#40234)
    
    
    
    ### Rationale for this change
    Fix an abort error on asof_join_benchmark run for lost an arg.
    
    ### What changes are included in this PR?
    Add the lost arg back.
    
    ### Are these changes tested?
    
    ### Are there any user-facing changes?
    No
    
    * GitHub Issue: #40233
    
    Authored-by: hugo.zhang <[email protected]>
    Signed-off-by: Antoine Pitrou <[email protected]>
---
 cpp/src/arrow/acero/asof_join_benchmark.cc | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/cpp/src/arrow/acero/asof_join_benchmark.cc 
b/cpp/src/arrow/acero/asof_join_benchmark.cc
index 600c230a3f..02116b09fc 100644
--- a/cpp/src/arrow/acero/asof_join_benchmark.cc
+++ b/cpp/src/arrow/acero/asof_join_benchmark.cc
@@ -131,9 +131,8 @@ static void AsOfJoinOverhead(benchmark::State& state) {
 // this generates the set of right hand tables to test on.
 void SetArgs(benchmark::internal::Benchmark* bench) {
   bench
-      ->ArgNames({"left_freq", "left_cols", "left_ids", "left_batch_size",
-                  "num_right_tables", "right_freq", "right_cols", "right_ids",
-                  "right_batch_size"})
+      ->ArgNames({"left_freq", "left_cols", "left_ids", "batch_size", 
"num_right_tables",
+                  "right_freq", "right_cols", "right_ids"})
       ->UseRealTime();
 
   int default_freq = 400;

Reply via email to