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

maplefu 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 ad6d6e7b50 GH-41192: [C++] Fix hashjoin benchmark failed at make 
utf8's random batches (#41195)
ad6d6e7b50 is described below

commit ad6d6e7b504f913ddb9a2c8be7008e05c54530ef
Author: ZhangHuiGui <106943008+zhanghui...@users.noreply.github.com>
AuthorDate: Mon Apr 15 13:12:05 2024 +0800

    GH-41192: [C++] Fix hashjoin benchmark failed at make utf8's random batches 
(#41195)
    
    
    
    ### Rationale for this change
    Fix hashjoin benchmark failed at make utf8's random batches.
    
    ### What changes are included in this PR?
    Add to_string for `min_length` and `max_length`'s value.
    
    ### Are these changes tested?
    Needn't
    
    ### Are there any user-facing changes?
    No
    
    * GitHub Issue: #41192
    
    Authored-by: ZhangHuiGui <hugo.zh...@openpie.com>
    Signed-off-by: mwish <maplewish...@gmail.com>
---
 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 993c0b9a70..ad1bd67cc8 100644
--- a/cpp/src/arrow/acero/hash_join_benchmark.cc
+++ b/cpp/src/arrow/acero/hash_join_benchmark.cc
@@ -83,8 +83,8 @@ class JoinBenchmark {
       build_metadata["null_probability"] = 
std::to_string(settings.null_percentage);
       build_metadata["min"] = std::to_string(min_build_value);
       build_metadata["max"] = std::to_string(max_build_value);
-      build_metadata["min_length"] = settings.var_length_min;
-      build_metadata["max_length"] = settings.var_length_max;
+      build_metadata["min_length"] = std::to_string(settings.var_length_min);
+      build_metadata["max_length"] = std::to_string(settings.var_length_max);
 
       std::unordered_map<std::string, std::string> probe_metadata;
       probe_metadata["null_probability"] = 
std::to_string(settings.null_percentage);

Reply via email to