Follow-up to #23696 (review comment).
The initial decoder / rg_plan / filter_installed / row_filter_context construction in ParquetOpener::open — the let (decoder, rg_plan, filter_installed, row_filter_context) = { ... } block in datafusion/datasource-parquet/src/opener/mod.rs — should be factored out into a helper returning a named struct (e.g. InitialDecoderState) to shorten open and make the setup easier to follow.
Suggested by @adriangb in #23696 (comment)
Follow-up to #23696 (review comment).
The initial decoder /
rg_plan/filter_installed/row_filter_contextconstruction inParquetOpener::open— thelet (decoder, rg_plan, filter_installed, row_filter_context) = { ... }block indatafusion/datasource-parquet/src/opener/mod.rs— should be factored out into a helper returning a named struct (e.g.InitialDecoderState) to shortenopenand make the setup easier to follow.Suggested by @adriangb in #23696 (comment)