use .any instead of filter/count

alamb · alamb · commit 2c6a23b58f60 · 2021-05-20T08:53:56.000-04:00
diff --git a/datafusion/src/physical_optimizer/pruning.rs b/datafusion/src/physical_optimizer/pruning.rs
@@ -244,11 +244,10 @@ impl<'a> StatisticsExpressionBuilder<'a> {
     }
 
     fn is_stat_column_missing(&self, statistics_type: StatisticsType) -> bool {
-        self.stat_column_req
+        !self
+            .stat_column_req
             .iter()
-            .filter(|(c, t, _f)| c == &self.column_name && t == &statistics_type)
-            .count()
-            == 0
+            .any(|(c, t, _f)| c == &self.column_name && t == &statistics_type)
     }
 
     fn stat_column_expr(

Original file line number	Diff line number	Diff line change
`@@ -244,11 +244,10 @@ impl<'a> StatisticsExpressionBuilder<'a> {`
`244`	`244`	`}`
`245`	`245`
`246`	`246`	`fn is_stat_column_missing(&self, statistics_type: StatisticsType) -> bool {`
`247`		`- self.stat_column_req`
	`247`	`+ !self`
	`248`	`+ .stat_column_req`
`248`	`249`	`.iter()`
`249`		`- .filter(\|(c, t, _f)\| c == &self.column_name && t == &statistics_type)`
`250`		`- .count()`
`251`		`- == 0`
	`250`	`+ .any(\|(c, t, _f)\| c == &self.column_name && t == &statistics_type)`
`252`	`251`	`}`
`253`	`252`
`254`	`253`	`fn stat_column_expr(`