Skip to content

Commit

Permalink
ARROW-4282: [Rust] builder benchmark is broken
Browse files Browse the repository at this point in the history
Author: Paddy Horan <[email protected]>

Closes apache#3422 from paddyhoran/bench-bug and squashes the following commits:

e8e7c9d <Paddy Horan> Bug fix in builder benchmark
  • Loading branch information
paddyhoran authored and kou committed Jan 20, 2019
1 parent 4d1ee0a commit 9c7853c
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions rust/arrow/benches/builder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ fn bench_primitive(c: &mut Criterion) {
b.iter(|| {
let mut builder = Int64Builder::new(64);
for _ in 0..NUM_BATCHES {
let _ = black_box(builder.push_slice(&data[..]));
let _ = black_box(builder.append_slice(&data[..]));
}
black_box(builder.finish());
})
Expand All @@ -62,7 +62,7 @@ fn bench_bool(c: &mut Criterion) {
b.iter(|| {
let mut builder = BooleanBuilder::new(64);
for _ in 0..NUM_BATCHES {
let _ = black_box(builder.push_slice(&data[..]));
let _ = black_box(builder.append_slice(&data[..]));
}
black_box(builder.finish());
})
Expand Down

0 comments on commit 9c7853c

Please sign in to comment.