fix: resolve clippy warnings and formatting issues (#33)

- Update BehaviorVersion::v2025_01_17 to ::latest() to fix deprecation warning
- Add missing limit field to FileScanTask test constructors in reader.rs
- Apply cargo fmt formatting fixes in row_delta.rs

Author: lukekim

crates/catalog/rest/src/middleware/sigv4.rs

@@ -106,7 +106,7 @@ impl Middleware for SigV4Middleware {
         let config = self
             .config
             .get_or_init(|| async {
-                let mut config_builder = aws_config::defaults(BehaviorVersion::v2025_01_17());
+                let mut config_builder = aws_config::defaults(BehaviorVersion::latest());
 
                 let region = Region::new(signing_region);
                 config_builder = config_builder.region(region.clone());

crates/iceberg/src/arrow/reader.rs

@@ -4225,6 +4225,7 @@ message schema {
                 project_field_ids: vec![1, 2],
                 predicate: None,
                 deletes: vec![],
+                limit: None,
                 partition: None,
                 partition_spec: None,
                 name_mapping: None,
@@ -4243,6 +4244,7 @@ message schema {
                 project_field_ids: vec![1, 2],
                 predicate: None,
                 deletes: vec![],
+                limit: None,
                 partition: None,
                 partition_spec: None,
                 name_mapping: None,
@@ -4261,6 +4263,7 @@ message schema {
                 project_field_ids: vec![1, 2],
                 predicate: None,
                 deletes: vec![],
+                limit: None,
                 partition: None,
                 partition_spec: None,
                 name_mapping: None,

crates/iceberg/src/transaction/row_delta.rs

@@ -219,10 +219,7 @@ mod tests {
             .unwrap()
     }
 
-    fn make_equality_delete_file(
-        table: &crate::table::Table,
-        path: &str,
-    ) -> crate::spec::DataFile {
+    fn make_equality_delete_file(table: &crate::table::Table, path: &str) -> crate::spec::DataFile {
         DataFileBuilder::default()
             .content(DataContentType::EqualityDeletes)
             .file_path(path.to_string())
@@ -236,10 +233,7 @@ mod tests {
             .unwrap()
     }
 
-    fn make_position_delete_file(
-        table: &crate::table::Table,
-        path: &str,
-    ) -> crate::spec::DataFile {
+    fn make_position_delete_file(table: &crate::table::Table, path: &str) -> crate::spec::DataFile {
         DataFileBuilder::default()
             .content(DataContentType::PositionDeletes)
             .file_path(path.to_string())
@@ -273,11 +267,9 @@ mod tests {
         let requirements = action_commit.take_requirements();
 
         // Check updates structure
-        assert!(
-            matches!((&updates[0], &updates[1]),
+        assert!(matches!((&updates[0], &updates[1]),
                 (TableUpdate::AddSnapshot { snapshot }, TableUpdate::SetSnapshotRef { reference, ref_name })
-                if snapshot.snapshot_id() == reference.snapshot_id && ref_name == MAIN_BRANCH)
-        );
+                if snapshot.snapshot_id() == reference.snapshot_id && ref_name == MAIN_BRANCH));
 
         // Check operation is Delete when only delete files present
         let new_snapshot = if let TableUpdate::AddSnapshot { snapshot } = &updates[0] {
@@ -453,9 +445,7 @@ mod tests {
             Err(e) => e.to_string(),
             Ok(_) => panic!("Expected error for V1 delete files"),
         };
-        assert!(
-            err_msg.contains("not supported in format version 1")
-        );
+        assert!(err_msg.contains("not supported in format version 1"));
     }
 
     #[tokio::test]