Skip to content

Commit e5fb84a

Browse files
authored
[benches/row_format] fix typo in array lengths (#7572)
# Which issue does this PR close? For some reason, some of the benchmarks create arrays of length `40960` .... that looks like a 0 too much to me =) # What changes are included in this PR? Changes 40960 to 4096 for some of the row_format benches # Are there any user-facing changes? No.
1 parent b10567f commit e5fb84a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

arrow/benches/row_format.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ fn do_bench(c: &mut Criterion, name: &str, cols: Vec<ArrayRef>) {
5757
}
5858

5959
fn bench_iter(c: &mut Criterion) {
60-
let col = create_string_view_array_with_len(40960, 0., 100, false);
60+
let col = create_string_view_array_with_len(4096, 0., 100, false);
6161
let converter = RowConverter::new(vec![SortField::new(col.data_type().clone())]).unwrap();
6262
let rows = converter
6363
.convert_columns(&[Arc::new(col) as ArrayRef])
@@ -109,8 +109,8 @@ fn row_bench(c: &mut Criterion) {
109109
let cols = vec![Arc::new(create_string_view_array_with_len(4096, 0., 30, false)) as ArrayRef];
110110
do_bench(c, "4096 string view(30, 0)", cols);
111111

112-
let cols = vec![Arc::new(create_string_view_array_with_len(40960, 0., 100, false)) as ArrayRef];
113-
do_bench(c, "40960 string view(100, 0)", cols);
112+
let cols = vec![Arc::new(create_string_view_array_with_len(4096, 0., 100, false)) as ArrayRef];
113+
do_bench(c, "4096 string view(100, 0)", cols);
114114

115115
let cols = vec![Arc::new(create_string_view_array_with_len(4096, 0.5, 100, false)) as ArrayRef];
116116
do_bench(c, "4096 string view(100, 0.5)", cols);

0 commit comments

Comments
 (0)