refine name

Author: ZENOTME

crates/iceberg/src/arrow/reader.rs

@@ -384,7 +384,7 @@ impl ArrowReader {
             });
 
             if column_map.len() != leaf_field_ids.len() {
-                let miss_field = leaf_field_ids
+                let missing_fields = leaf_field_ids
                     .iter()
                     .filter(|field_id| !column_map.contains_key(field_id))
                     .collect::<Vec<_>>();
@@ -397,7 +397,7 @@ impl ArrowReader {
                 )
                 .with_context("column_map", format! {"{:?}", column_map})
                 .with_context("field_ids", format! {"{:?}", leaf_field_ids})
-                .with_context("miss_field", format! {"{:?}", miss_field}));
+                .with_context("missing_fields", format! {"{:?}", missing_fields}));
             }
 
             let mut indices = vec![];

crates/iceberg/src/spec/datatypes.rs

@@ -36,7 +36,7 @@ use crate::spec::datatypes::_decimal::{MAX_PRECISION, REQUIRED_LENGTH};
 use crate::spec::PrimitiveLiteral;
 
 /// Field name for list type.
-pub const LIST_FILED_NAME: &str = "element";
+pub const LIST_FIELD_NAME: &str = "element";
 /// Field name for map type's key.
 pub const MAP_KEY_FIELD_NAME: &str = "key";
 /// Field name for map type's value.
@@ -635,9 +635,9 @@ impl NestedField {
     /// Construct list type's element field.
     pub fn list_element(id: i32, field_type: Type, required: bool) -> Self {
         if required {
-            Self::required(id, LIST_FILED_NAME, field_type)
+            Self::required(id, LIST_FIELD_NAME, field_type)
         } else {
-            Self::optional(id, LIST_FILED_NAME, field_type)
+            Self::optional(id, LIST_FIELD_NAME, field_type)
         }
     }
 

crates/iceberg/src/spec/schema.rs

@@ -30,7 +30,7 @@ use super::NestedField;
 use crate::error::Result;
 use crate::expr::accessor::StructAccessor;
 use crate::spec::datatypes::{
-    ListType, MapType, NestedFieldRef, PrimitiveType, StructType, Type, LIST_FILED_NAME,
+    ListType, MapType, NestedFieldRef, PrimitiveType, StructType, Type, LIST_FIELD_NAME,
     MAP_KEY_FIELD_NAME, MAP_VALUE_FIELD_NAME,
 };
 use crate::{ensure_data_valid, Error, ErrorKind};
@@ -774,7 +774,7 @@ impl SchemaVisitor for IndexByName {
     }
 
     fn list(&mut self, list: &ListType, _value: Self::T) -> Result<Self::T> {
-        self.add_field(LIST_FILED_NAME, list.element_field.id)
+        self.add_field(LIST_FIELD_NAME, list.element_field.id)
     }
 
     fn map(&mut self, map: &MapType, _key_value: Self::T, _value: Self::T) -> Result<Self::T> {

crates/integration_tests/tests/scan_all_type.rs

@@ -30,7 +30,7 @@ use arrow_schema::{DataType, Field, Fields};
 use futures::TryStreamExt;
 use iceberg::arrow::{DEFAULT_MAP_FIELD_NAME, UTC_TIME_ZONE};
 use iceberg::spec::{
-    ListType, MapType, NestedField, PrimitiveType, Schema, StructType, Type, LIST_FILED_NAME,
+    ListType, MapType, NestedField, PrimitiveType, Schema, StructType, Type, LIST_FIELD_NAME,
     MAP_KEY_FIELD_NAME, MAP_VALUE_FIELD_NAME,
 };
 use iceberg::transaction::Transaction;
@@ -240,7 +240,7 @@ async fn test_scan_all_type() {
     ]);
     let col16 = {
         let mut builder = ListBuilder::new(Int32Builder::new()).with_field(Arc::new(
-            Field::new(LIST_FILED_NAME, DataType::Int32, false).with_metadata(HashMap::from([(
+            Field::new(LIST_FIELD_NAME, DataType::Int32, false).with_metadata(HashMap::from([(
                 PARQUET_FIELD_ID_META_KEY.to_string(),
                 20.to_string(),
             )])),