adriangb commented on PR #10549: URL: https://github.com/apache/arrow-rs/pull/10549#issuecomment-5183396243
Benchmarking this against https://github.com/apache/arrow-rs/pull/10550 as the baseline, so the only difference between the two sides is the block-wise prefix scan itself — both refs carry the new `small_string_*` benches. ```yaml run benchmark arrow_writer baseline: ref: a5444d2330424f563277f492a56e96a4dbf6ca8e changed: ref: 2404f178af005afd1f0c4ddef29311d561aaf700 env: BENCH_FILTER: small_string_ ``` The `plain` variant of each group is a control: `PLAIN` goes through `FallbackEncoderImpl::Plain` and never calls `common_prefix_length`, so it runs identical code on both sides and should come out flat. `small_string_distinct` should also be flat — its values differ from byte 0, so there is no shared prefix to scan. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
