Repository navigation
Deprecate PruningPredicate::try_new in favor of PruningPredicateBuilder #24128
Description
Follow-up to #24074, per @alamb's suggestion.
#24074 introduced PruningPredicateBuilder (.with_file_schema(...), .with_error_counter(...), .with_max_in_list_size(...), .build(...)/.try_build(...)) as the extensible entry point for constructing a PruningPredicate.
As a follow-up we should:
- Point people at
PruningPredicateBuilderfrom thePruningPredicate::try_newdoc comment. - Deprecate
PruningPredicate::try_new(#[deprecated]) and migrate its call sites to the builder. - Move the construction logic into the builder so there is a single place to add future parameters.
Non-blocking cleanup — no behavior change.
Metadata
Metadata
Assignees
Labels
No labels
Activity
goutamadwant commentedon Aug 6, 2026
take
Merge branch 'main' into apachegh-24128-pruning-predicate-builder
goutamadwant commentedon Aug 6, 2026
Opened #24129 . This moves pruning predicate construction into
PruningPredicateBuilder, deprecatesPruningPredicate::try_new, and migrates DataFusion's internal call sites and examples. The deprecated constructor remains as a compatibility wrapper with unchanged behavior.Merge branch 'main' into apachegh-24128-pruning-predicate-builder
refactor(pruning): deprecate PruningPredicate::try_new (apache#24129)
refactor(pruning): deprecate PruningPredicate::try_new (apache#24129)