Skip to content

Commit 5453c57

Browse files
committed
Fix clippy
1 parent 559321a commit 5453c57

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

crates/iceberg/src/arrow/reader.rs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -233,8 +233,7 @@ impl ArrowReader {
233233
let mut column_indices = collector
234234
.field_ids
235235
.iter()
236-
.map(|field_id| field_id_map.get(field_id).cloned())
237-
.flatten()
236+
.filter_map(|field_id| field_id_map.get(field_id).cloned())
238237
.collect::<Vec<_>>();
239238

240239
column_indices.sort();
@@ -580,7 +579,7 @@ impl<'a> BoundPredicateVisitor for PredicateConverter<'a> {
580579
reference: &BoundReference,
581580
_predicate: &BoundPredicate,
582581
) -> Result<Box<PredicateResult>> {
583-
if let Some(_) = self.bound_reference(reference) {
582+
if self.bound_reference(reference).is_some() {
584583
self.build_always_true()
585584
} else {
586585
// A missing column, treating it as null.
@@ -593,7 +592,7 @@ impl<'a> BoundPredicateVisitor for PredicateConverter<'a> {
593592
reference: &BoundReference,
594593
_predicate: &BoundPredicate,
595594
) -> Result<Box<PredicateResult>> {
596-
if let Some(_) = self.bound_reference(reference) {
595+
if self.bound_reference(reference).is_some() {
597596
self.build_always_false()
598597
} else {
599598
// A missing column, treating it as null.

0 commit comments

Comments
 (0)