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);