Skip to content

Commit 71dd48e

Browse files
authored
[Variant] Add variant_kernels benchmark (#7944)
# Which issue does this PR close? We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. - Part of #7911 - Part of #6736 - Follow on to #7905 # Rationale for this change I wrote benchmark some changes to the json decoder in #7911 but they are non trivial. To keep #7911 easier to review I have pulled the benchmarks out to their own PR # What changes are included in this PR? 1. Add new json benchmark 2. Include the `variant_get` benchmark added in #7919 by @Samyak2 # Are these changes tested? I tested them manually and clippy CI coverage ensures they compile # Are there any user-facing changes? No these are only benchmarks
1 parent a5afda2 commit 71dd48e

File tree

3 files changed

+367
-61
lines changed

3 files changed

+367
-61
lines changed

parquet-variant-compute/Cargo.toml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,11 @@ name = "parquet_variant_compute"
4141
bench = false
4242

4343
[dev-dependencies]
44+
rand = "0.9.1"
4445
criterion = { version = "0.6", default-features = false }
45-
rand = { version = "0.9.1" }
46+
4647

4748
[[bench]]
48-
name = "variant_get"
49+
name = "variant_kernels"
4950
harness = false
51+

parquet-variant-compute/benches/variant_get.rs

Lines changed: 0 additions & 59 deletions
This file was deleted.

0 commit comments

Comments
 (0)