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 <[email protected]>
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 <[email protected]>
Signed-off-by: mwish <[email protected]>
---
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);