Skip to content

feat(parquet): add struct-column writer benchmarks#9679

Merged
alamb merged 1 commit intoapache:mainfrom
HippoBaro:sparse_benchmarks
Apr 9, 2026
Merged

feat(parquet): add struct-column writer benchmarks#9679
alamb merged 1 commit intoapache:mainfrom
HippoBaro:sparse_benchmarks

Conversation

@HippoBaro
Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

#9653 introduces optimizations related to non-null uniform workloads. This adds benchmarks so we can quantify them.

What changes are included in this PR?

Add three new benchmark cases to the arrow_writer benchmark suite for evaluating write performance on struct columns at varying null densities:

  • struct_non_null: a nullable struct with 0% null rows and non-nullable primitive children;
  • struct_sparse_99pct_null: a nullable struct with 99% null rows, exercising null batching through one level of struct nesting;
  • struct_all_null: a nullable struct with 100% null rows, exercising the uniform-null path through struct nesting.

Baseline results (Apple M1 Max):

  struct_non_null/default              29.9 ms
  struct_non_null/parquet_2            38.2 ms
  struct_non_null/zstd_parquet_2       50.9 ms
  struct_sparse_99pct_null/default      7.2 ms
  struct_sparse_99pct_null/parquet_2    7.3 ms
  struct_sparse_99pct_null/zstd_p2      8.1 ms
  struct_all_null/default              83.3 µs
  struct_all_null/parquet_2            82.5 µs
  struct_all_null/zstd_parquet_2      106.6 µs

Are these changes tested?

N/A

Are there any user-facing changes?

None

@github-actions github-actions bot added the parquet Changes to the parquet crate label Apr 8, 2026
Add three new benchmark cases to the arrow_writer benchmark suite for
evaluating write performance on struct columns at varying null
densities:

- `struct_non_null`: a nullable struct with 0% null rows and
  non-nullable primitive children;
- `struct_sparse_99pct_null`: a nullable struct with 99% null rows,
  exercising null batching through one level of struct nesting;
- `struct_all_null`: a nullable struct with 100% null rows, exercising
  the uniform-null path through struct nesting.

  Baseline results (Apple M1 Max):

  struct_non_null/default              29.9 ms
  struct_non_null/parquet_2            38.2 ms
  struct_non_null/zstd_parquet_2       50.9 ms
  struct_sparse_99pct_null/default      7.2 ms
  struct_sparse_99pct_null/parquet_2    7.3 ms
  struct_sparse_99pct_null/zstd_p2      8.1 ms
  struct_all_null/default              83.3 µs
  struct_all_null/parquet_2            82.5 µs
  struct_all_null/zstd_parquet_2      106.6 µs

Signed-off-by: Hippolyte Barraud <hippolyte.barraud@datadoghq.com>
@alamb alamb merged commit adf9308 into apache:main Apr 9, 2026
16 checks passed
@alamb
Copy link
Copy Markdown
Contributor

alamb commented Apr 9, 2026

Thank you @HippoBaro

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants