One DuckDB fix

Signed-off-by: Adam Gutglick <[email protected]>

Author: AdamGS

benchmarks/ddb-bench/src/lib.rs

@@ -49,22 +49,27 @@ impl DuckClient {
         delete_database: bool,
         threads: Option<usize>,
     ) -> Result<Self> {
-        let base_path = data_url
-            .to_file_path()
-            .map_err(|_| anyhow::anyhow!("Invalid file URL: {}", data_url))?;
-        let dir = base_path.join(format.name());
-        std::fs::create_dir_all(&dir)?;
-        let db_path = dir.join("duckdb.db");
-        if delete_database && db_path.exists() {
-            std::fs::remove_file(&db_path)?;
-        }
+        let db_path = if data_url.scheme() != "file" {
+            None
+        } else {
+            let base_path = data_url
+                .to_file_path()
+                .map_err(|_| anyhow::anyhow!("Invalid file URL: {}", data_url))?;
+            let dir = base_path.join(format.name());
+            std::fs::create_dir_all(&dir)?;
+            let db_path = dir.join("duckdb.db");
+            if delete_database && db_path.exists() {
+                std::fs::remove_file(&db_path)?;
+            }
+            Some(db_path)
+        };
 
-        let (db, connection) = Self::open_and_setup_database(Some(db_path.clone()), threads)?;
+        let (db, connection) = Self::open_and_setup_database(db_path.clone(), threads)?;
 
         Ok(Self {
             db,
             connection,
-            db_path: Some(db_path),
+            db_path,
             threads,
         })
     }

vortex-bench/src/benchmark.rs

@@ -11,6 +11,7 @@ use crate::BenchmarkDataset;
 use crate::Format;
 
 /// Specification for a table in a benchmark dataset.
+#[derive(Debug)]
 pub struct TableSpec {
     pub name: &'static str,
     pub schema: Option<Schema>,