fix: Fix remaining format string warnings in CLI crate

- Fix 12 clippy::uninlined_format_args warnings in datafusion-postgres-cli
- Update all format! macros to use modern inline variable syntax
- Includes table registration error messages and println statements
- Now passes clippy --workspace --all-targets --all-features -- -D warnings

Author: iPeluwa

datafusion-postgres-cli/src/main.rs

@@ -111,7 +111,7 @@ impl Opt {
                     }
                 }
                 Err(e) => {
-                    return Err(format!("Failed to load directory {}: {}", directory, e).into());
+                    return Err(format!("Failed to load directory {directory}: {e}").into());
                 }
             }
         }
@@ -128,17 +128,17 @@ async fn setup_session_context(
         session_context
             .register_csv(table_name, table_path, CsvReadOptions::default())
             .await
-            .map_err(|e| format!("Failed to register CSV table '{}': {}", table_name, e))?;
-        println!("Loaded {} as table {}", table_path, table_name);
+            .map_err(|e| format!("Failed to register CSV table '{table_name}': {e}"))?;
+        println!("Loaded {table_path} as table {table_name}");
     }
 
     // Register JSON tables
     for (table_name, table_path) in opts.json_tables.iter().map(|s| parse_table_def(s.as_ref())) {
         session_context
             .register_json(table_name, table_path, NdJsonReadOptions::default())
             .await
-            .map_err(|e| format!("Failed to register JSON table '{}': {}", table_name, e))?;
-        println!("Loaded {} as table {}", table_path, table_name);
+            .map_err(|e| format!("Failed to register JSON table '{table_name}': {e}"))?;
+        println!("Loaded {table_path} as table {table_name}");
     }
 
     // Register Arrow tables
@@ -150,8 +150,8 @@ async fn setup_session_context(
         session_context
             .register_arrow(table_name, table_path, ArrowReadOptions::default())
             .await
-            .map_err(|e| format!("Failed to register Arrow table '{}': {}", table_name, e))?;
-        println!("Loaded {} as table {}", table_path, table_name);
+            .map_err(|e| format!("Failed to register Arrow table '{table_name}': {e}"))?;
+        println!("Loaded {table_path} as table {table_name}");
     }
 
     // Register Parquet tables
@@ -163,17 +163,17 @@ async fn setup_session_context(
         session_context
             .register_parquet(table_name, table_path, ParquetReadOptions::default())
             .await
-            .map_err(|e| format!("Failed to register Parquet table '{}': {}", table_name, e))?;
-        println!("Loaded {} as table {}", table_path, table_name);
+            .map_err(|e| format!("Failed to register Parquet table '{table_name}': {e}"))?;
+        println!("Loaded {table_path} as table {table_name}");
     }
 
     // Register Avro tables
     for (table_name, table_path) in opts.avro_tables.iter().map(|s| parse_table_def(s.as_ref())) {
         session_context
             .register_avro(table_name, table_path, AvroReadOptions::default())
             .await
-            .map_err(|e| format!("Failed to register Avro table '{}': {}", table_name, e))?;
-        println!("Loaded {} as table {}", table_path, table_name);
+            .map_err(|e| format!("Failed to register Avro table '{table_name}': {e}"))?;
+        println!("Loaded {table_path} as table {table_name}");
     }
 
     // Register pg_catalog
@@ -202,7 +202,7 @@ async fn main() -> Result<(), Box<dyn std::error::Error>> {
 
     serve(Arc::new(session_context), &server_options)
         .await
-        .map_err(|e| format!("Failed to run server: {}", e))?;
+        .map_err(|e| format!("Failed to run server: {e}"))?;
 
     Ok(())
 }