Update datafusion/expr/src/expr_fn.rs

thinkharderdev · alamb · web-flow · commit de4685018394 · 2022-09-12T19:25:13.000-04:00
Co-authored-by: Andrew Lamb &lt;andrew@nerdnetworks.org&gt;
diff --git a/datafusion/expr/src/expr_fn.rs b/datafusion/expr/src/expr_fn.rs
@@ -458,8 +458,7 @@ pub fn combine_filters_disjunctive(filters: &[Expr]) -> Option<Expr> {
     }
     let combined_filter = filters
         .iter()
-        .skip(1)
-        .fold(filters[0].clone(), |acc, filter| or(acc, filter.clone()));
+        .reduce(|acc, filter| or(acc, filter.clone()));
     Some(combined_filter)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -458,8 +458,7 @@ pub fn combine_filters_disjunctive(filters: &[Expr]) -> Option<Expr> {`
`458`	`458`	`}`
`459`	`459`	`let combined_filter = filters`
`460`	`460`	`.iter()`
`461`		`- .skip(1)`
`462`		`- .fold(filters[0].clone(), \|acc, filter\| or(acc, filter.clone()));`
	`461`	`+ .reduce(\|acc, filter\| or(acc, filter.clone()));`
`463`	`462`	`Some(combined_filter)`
`464`	`463`	`}`
`465`	`464`