geoffreyclaude opened a new pull request, #21648:
URL: https://github.com/apache/datafusion/pull/21648

   ## Which issue does this PR close?
   
   - Part of #19241.
   - Split out from #19390 following reviewer feedback to land the benchmark 
harness independently.
   
   ## Rationale for this change
   
   `#19390` bundles benchmark additions together with a large series of `IN 
LIST` implementation optimizations. In the review discussion, the maintainers 
suggested splitting out the benchmark-only work first so the follow-up 
optimization PRs can be evaluated against a stable, focused benchmark corpus.
   
   This PR contains only the benchmark additions. It does not change `InList` 
execution behavior.
   
   ## What changes are included in this PR?
   
   - Adds `datafusion/physical-expr/benches/in_list_strategy.rs`
   - Adds focused microbenchmarks for the filter strategies and threshold 
boundaries explored in the follow-up optimization PRs
   - Keeps the existing `benches/in_list.rs` benchmarks intact for historical 
comparison
   - Registers the new benchmark target in `datafusion/physical-expr/Cargo.toml`
   
   ## Are these changes tested?
   
   Yes. I validated this split with:
   
   - `cargo fmt --all`
   - `cargo clippy -p datafusion-physical-expr --all-targets --all-features -- 
-D warnings`
   - `cargo test -p datafusion-physical-expr in_list --lib`
   
   ## Are there any user-facing changes?
   
   No user-facing changes. This PR only adds benchmark coverage.
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to