You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by jo...@apache.org on 2022/07/12 20:59:30 UTC

[arrow] branch master updated: ARROW-17059: [C++] Fix expression benchmark (#13584)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new b87e0c1dad ARROW-17059: [C++] Fix expression benchmark (#13584)
b87e0c1dad is described below

commit b87e0c1dad77c2d95fb979bce831a57d6ae60daa
Author: Sasha Krassovsky <kr...@gmail.com>
AuthorDate: Tue Jul 12 12:59:16 2022 -0800

    ARROW-17059: [C++] Fix expression benchmark (#13584)
    
    Authored-by: Sasha Krassovsky <kr...@gmail.com>
    Signed-off-by: Jonathan Keane <jk...@gmail.com>
---
 cpp/src/arrow/compute/exec/expression_benchmark.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/cpp/src/arrow/compute/exec/expression_benchmark.cc b/cpp/src/arrow/compute/exec/expression_benchmark.cc
index 70aa509d2e..debd228498 100644
--- a/cpp/src/arrow/compute/exec/expression_benchmark.cc
+++ b/cpp/src/arrow/compute/exec/expression_benchmark.cc
@@ -80,8 +80,8 @@ static void ExecuteScalarExpressionOverhead(benchmark::State& state, Expression
   });
   std::vector<ExecBatch> inputs(num_batches);
   for (auto& batch : inputs) {
-    batch = ExecBatch({Datum(ConstantArrayGenerator::Int64(rows_per_batch, 5))},
-                      /*length=*/1);
+    batch = ExecBatch({Datum(ConstantArrayGenerator::Int64(rows_per_batch, /*value=*/5))},
+                      /*length=*/rows_per_batch);
   }
 
   ASSIGN_OR_ABORT(auto bound, expr.Bind(*dataset_schema));