File tree 3 files changed +11
-8
lines changed 3 files changed +11
-8
lines changed Original file line number Diff line number Diff line change @@ -612,10 +612,9 @@ impl FileSource for ParquetSource {
612
612
let pushdown_filters = table_pushdown_enabled || config_pushdown_enabled;
613
613
614
614
let mut source = self . clone ( ) ;
615
- let filters = PredicateSupports :: new_with_supported_check (
616
- filters,
617
- |filter| pushdown_filters && can_expr_be_pushed_down_with_schemas ( filter, & file_schema) ,
618
- ) ;
615
+ let filters = PredicateSupports :: new_with_supported_check ( filters, |filter| {
616
+ pushdown_filters && can_expr_be_pushed_down_with_schemas ( filter, & file_schema)
617
+ } ) ;
619
618
if filters. is_all_unsupported ( ) {
620
619
// No filters can be pushed down, so we can just return the remaining filters
621
620
// and avoid replacing the source in the physical plan.
Original file line number Diff line number Diff line change @@ -535,11 +535,11 @@ pub trait ExecutionPlan: Debug + DisplayAs + Send + Sync {
535
535
/// and [`ChildPushdownResult::self_filters`] is discarded.
536
536
///
537
537
/// The default implementation is a no-op that passes the result of pushdown from the children to its parent.
538
- ///
538
+ ///
539
539
/// When returning filters via [`FilterPushdownPropagation`] the order of the filters need not match
540
540
/// the order they were passed in via `child_pushdown_result`, but preserving the order may be beneficial
541
541
/// for debugging and reasoning about the resulting plans so it is recommended to preserve the order.
542
- ///
542
+ ///
543
543
/// There are various helper methods to make implementing this method easier, see:
544
544
/// - [`FilterPushdownPropagation::unsupported`]: to indicate that the node does not support filter pushdown at all.
545
545
/// - [`FilterPushdownPropagation::transparent`]: to indicate that the node supports filter pushdown but does not involve itself in it,
Original file line number Diff line number Diff line change @@ -166,12 +166,16 @@ impl PredicateSupports {
166
166
167
167
/// Check if all filters are supported.
168
168
pub fn is_all_supported ( & self ) -> bool {
169
- self . 0 . iter ( ) . all ( |f| matches ! ( f, PredicateSupport :: Supported ( _) ) )
169
+ self . 0
170
+ . iter ( )
171
+ . all ( |f| matches ! ( f, PredicateSupport :: Supported ( _) ) )
170
172
}
171
173
172
174
/// Check if all filters are unsupported.
173
175
pub fn is_all_unsupported ( & self ) -> bool {
174
- self . 0 . iter ( ) . all ( |f| matches ! ( f, PredicateSupport :: Unsupported ( _) ) )
176
+ self . 0
177
+ . iter ( )
178
+ . all ( |f| matches ! ( f, PredicateSupport :: Unsupported ( _) ) )
175
179
}
176
180
}
177
181
You can’t perform that action at this time.
0 commit comments