Fix lint, fix integration test

src/sql/db_connection_pool/sqlitepool.rs

@@ -157,7 +157,7 @@ impl SqliteConnectionPool {
 
     pub async fn setup(&self) -> Result<()> {
         let conn = self.conn.clone();
-        let busy_timeout = self.busy_timeout.clone();
+        let busy_timeout = self.busy_timeout;
 
         // these configuration options are only applicable for file-mode databases
         if self.mode == Mode::File {
@@ -226,7 +226,7 @@ impl SqliteConnectionPool {
                 mode: self.mode,
                 path: Arc::clone(&self.path),
                 attach_databases: self.attach_databases.clone(),
-                busy_timeout: self.busy_timeout.clone(),
+                busy_timeout: self.busy_timeout,
             }),
             Mode::File => {
                 let attach_databases = if self.attach_databases.is_empty() {

src/sqlite.rs

@@ -154,10 +154,10 @@ impl SqliteTableProviderFactory {
                     }
                     .build()
                 })?;
-                return Ok(Duration::from_millis(result));
+                Ok(Duration::from_millis(result))
             }
-            None => return Ok(Duration::from_millis(5000)),
-        };
+            None => Ok(Duration::from_millis(5000)),
+        }
     }
 
     pub async fn get_or_init_instance(

tests/sqlite/mod.rs

@@ -21,10 +21,14 @@ async fn arrow_sqlite_round_trip(
     tracing::debug!("Running tests on {table_name}");
     let ctx = SessionContext::new();
 
-    let pool = SqliteConnectionPoolFactory::new(":memory:", Mode::Memory)
-        .build()
-        .await
-        .expect("Sqlite connection pool to be created");
+    let pool = SqliteConnectionPoolFactory::new(
+        ":memory:",
+        Mode::Memory,
+        std::time::Duration::from_millis(5000),
+    )
+    .build()
+    .await
+    .expect("Sqlite connection pool to be created");
 
     let conn = pool
         .connect()