style: fix clippy

GreptimeTeam · Apr 30, 2024 · e7e5dc6 · e7e5dc6
1 parent ff98cd5
commit e7e5dc6
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/src/mito2/src/sst/parquet/format.rs b/src/mito2/src/sst/parquet/format.rs
@@ -140,7 +140,7 @@ impl ReadFormat {
 
         // Maps column id of a projected field to its index in SST.
         let mut projected_field_id_index: Vec<_> = column_ids
-            .into_iter()
+            .iter()
             .filter_map(|column_id| {
                 // Only apply projection to fields.
                 field_id_to_index

diff --git a/src/mito2/src/sst/parquet/reader.rs b/src/mito2/src/sst/parquet/reader.rs
@@ -162,10 +162,7 @@ impl ParquetReaderBuilder {
         let region_meta = Arc::new(Self::get_region_metadata(&file_path, key_value_meta)?);
         // Lists all column ids to read, we always use the expected metadata if possible.
         let column_ids = self.projection.clone().unwrap_or_else(|| {
-            let expected_meta = self
-                .expected_metadata
-                .as_ref()
-                .unwrap_or_else(|| &region_meta);
+            let expected_meta = self.expected_metadata.as_ref().unwrap_or(&region_meta);
             expected_meta
                 .column_metadatas
                 .iter()
@@ -531,7 +528,7 @@ impl ReaderState {
     fn metrics(&self) -> &Metrics {
         match self {
             ReaderState::Readable(reader) => &reader.metrics,
-            ReaderState::Exhausted(m) => &m,
+            ReaderState::Exhausted(m) => m,
         }
     }
 }