Skip to content

Commit 68cd67b

Browse files
authored
chore: remove unused dependency (apache#823)
* chore: remove arrow2 benchmark * chore: remove unused dep
1 parent ed77904 commit 68cd67b

File tree

4 files changed

+7
-272
lines changed

4 files changed

+7
-272
lines changed

Cargo.lock

+5-159
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

benchmarks/benches/bench.rs

+1-30
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,10 @@
1-
// Copyright 2022 CeresDB Project Authors. Licensed under Apache-2.0.
1+
// Copyright 2022-2023 CeresDB Project Authors. Licensed under Apache-2.0.
22

33
//! Benchmarks
44
55
use std::sync::Once;
66

77
use benchmarks::{
8-
arrow2_bench::Arrow2Bench,
98
config::{self, BenchConfig},
109
merge_memtable_bench::MergeMemTableBench,
1110
merge_sst_bench::MergeSstBench,
@@ -171,33 +170,6 @@ fn bench_merge_memtable(c: &mut Criterion) {
171170
group.finish();
172171
}
173172

174-
fn bench_arrow2_iter(b: &mut Bencher<'_>, bench: &Arrow2Bench) {
175-
b.iter(|| bench.run_bench())
176-
}
177-
178-
fn bench_arrow2(c: &mut Criterion) {
179-
let config = init_bench();
180-
181-
let mut group = c.benchmark_group("read_arrow2");
182-
183-
group.measurement_time(config.sst_bench.bench_measurement_time.0);
184-
group.sample_size(config.sst_bench.bench_sample_size);
185-
186-
let mut bench = Arrow2Bench::new(config.sst_bench);
187-
188-
for i in 0..bench.num_benches() {
189-
bench.init_for_bench(i);
190-
191-
group.bench_with_input(
192-
BenchmarkId::new("read_arrow2", format!("{}/{}", bench.sst_file_name, i)),
193-
&bench,
194-
bench_arrow2_iter,
195-
);
196-
}
197-
198-
group.finish();
199-
}
200-
201173
fn bench_wal_write_iter(b: &mut Bencher<'_>, bench: &WalWriteBench) {
202174
b.iter(|| bench.run_bench())
203175
}
@@ -229,7 +201,6 @@ criterion_group!(
229201
bench_merge_sst,
230202
bench_scan_memtable,
231203
bench_merge_memtable,
232-
bench_arrow2,
233204
bench_wal_write,
234205
);
235206

0 commit comments

Comments
 (0)