zhuqi-lucas opened a new pull request, #24572: URL: https://github.com/apache/datafusion/pull/24572
## Which issue does this PR close? - Closes #24286. ## Rationale for this change Split out from #23696 (per @adriangb's [decomposition proposal](https://github.com/apache/datafusion/pull/23696#issuecomment-5346992228), PR 3). The decoder-setup block in the opener returns a bare `(decoder, rg_plan, has_row_selection)` tuple. Giving it a named `InitialDecoderState` struct keeps the values carried out of the block self-documenting, and means the follow-up feature PR (#23696) adds fields to a named struct instead of reshaping a positional tuple. ## What changes are included in this PR? - `push_decoder.rs`: new `pub(crate) struct InitialDecoderState { decoder, rg_plan, has_row_selection }`. - `opener/mod.rs`: the decoder-setup block returns `InitialDecoderState { .. }` and is destructured back into the same locals. Pure refactor, no behavior change. ## Are these changes tested? Covered by the existing `dynamic_row_group_pruning` integration tests (they drive the opener path); all pass. `clippy --all-targets --all-features -D warnings` is clean. ## Are there any user-facing changes? No. -- 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]
