Skip to content

Parametrize RDataFrame snapshot benchmarks over compression levels - #308

Merged
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:rdfsnapshot-compression-levels
Sep 3, 2026
Merged

Parametrize RDataFrame snapshot benchmarks over compression levels#308
guitargeek merged 1 commit into
root-project:masterfrom
guitargeek:rdfsnapshot-compression-levels

Conversation

@guitargeek

Copy link
Copy Markdown
Contributor

Each compression algorithm is now benchmarked at levels 1, 6 and 9 instead of a single hardcoded level, and the benchmarks report the compressed file size and the throughput of uncompressed input bytes.

The benchmarked dataset is also made big enough for the compression to dominate over the fixed snapshot overheads (10 million doubles instead of 100 thousand). The input data is generated only once, so that unlike before, the random number generation does not contribute to the measured time. The output file is now written to RB_TEMP_FS and removed afterwards.

This implements the improvements proposed for these benchmarks in PR #225 on top of the current state of the repository.

Each compression algorithm is now benchmarked at levels 1, 6 and 9
instead of a single hardcoded level, and the benchmarks report the
compressed file size and the throughput of uncompressed input bytes.

The benchmarked dataset is also made big enough for the compression to
dominate over the fixed snapshot overheads (10 million doubles instead
of 100 thousand). The input data is generated only once, so that unlike
before, the random number generation does not contribute to the
measured time. The output file is now written to RB_TEMP_FS and removed
afterwards.

This implements the improvements proposed for these benchmarks in
PR root-project#225 on top of the current state of the repository.
@guitargeek
guitargeek merged commit f2bec94 into root-project:master Sep 3, 2026
1 check passed
@guitargeek
guitargeek deleted the rdfsnapshot-compression-levels branch September 3, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant