fix: CI check failures for spiceai-0.9.0 merge

- Remove unused direct aws-sdk-sts dependency from iceberg-catalog-rest
  (transitively provided by aws-config), fixing cargo-machete check
- Only display limit in IcebergTableScan when set, fixing sqllogictest
  mismatches in binary and boolean predicate pushdown tests

Author: lukekim

Cargo.lock

@@ -34,7 +34,6 @@ async-trait = { workspace = true }
 aws-config = { workspace = true, optional = true }
 aws-sigv4 = { workspace = true, optional = true }
 aws-credential-types = { workspace = true, optional = true }
-aws-sdk-sts = { workspace = true, optional = true }
 chrono = { workspace = true }
 http = { workspace = true }
 iceberg = { workspace = true }
@@ -56,4 +55,4 @@ tokio = { workspace = true }
 wiremock = "=0.6.4"
 [features]
 default = ["sigv4"]
-sigv4 = ["aws-config", "aws-sigv4", "aws-credential-types", "aws-sdk-sts"]
+sigv4 = ["aws-config", "aws-sigv4", "aws-credential-types"]

crates/catalog/rest/Cargo.toml

@@ -190,15 +190,18 @@ impl DisplayAs for IcebergTableScan {
     ) -> std::fmt::Result {
         write!(
             f,
-            "IcebergTableScan projection:[{}] predicate:[{}] limit:[{}]",
+            "IcebergTableScan projection:[{}] predicate:[{}]",
             self.projection
                 .clone()
                 .map_or(String::new(), |v| v.join(",")),
             self.predicates
                 .clone()
                 .map_or(String::from(""), |p| format!("{p}")),
-            self.limit.map_or(String::from(""), |p| format!("{p}")),
-        )
+        )?;
+        if let Some(limit) = self.limit {
+            write!(f, " limit:[{limit}]")?;
+        }
+        Ok(())
     }
 }