Fix styling and documentation

Author: crystalxyz

README.md

@@ -27,7 +27,7 @@ let ctx = SessionContext::with_state(state);
 - Flight SQL
 - ODBC
 
-## Examples
+## Examples (in Rust)
 
 Run the included examples to see how to use the table providers:
 
@@ -45,6 +45,7 @@ cargo run --example duckdb_function --features duckdb
 ### SQLite
 
 ```bash
+# Run from repo folder
 cargo run --example sqlite --features sqlite
 ```
 
@@ -69,10 +70,8 @@ EOF
 ```
 
 ```bash
-# If running from core/ folder
-cargo run --example postgres --features postgres
-# If running from main repo folder
-cargo run -p datafusion-table-providers --example postgres --no-default-features --features postgres
+# Run from repo folder
+cargo run -p datafusion-table-providers --example postgres --features postgres
 
 ```
 
@@ -97,10 +96,8 @@ EOF
 ```
 
 ```bash
-# If run from core/ folder
-cargo run --example mysql --features mysql
-# If run from main repo folder
-cargo run -p datafusion-table-providers --example mysql --no-default-features --features mysql
+# Run from repo folder
+cargo run -p datafusion-table-providers --example mysql --features mysql
 ```
 
 ### Flight SQL
@@ -111,7 +108,8 @@ brew install roapi
 # cargo install --locked --git https://github.com/roapi/roapi --branch main --bins roapi
 roapi -t taxi=https://d37ci6vzurychx.cloudfront.net/trip-data/yellow_tripdata_2024-01.parquet &
 
-cargo run -p datafusion-table-providers --example flight-sql --no-default-features --features flight
+# Run from repo folder
+cargo run -p datafusion-table-providers --example flight-sql --features flight
 ```
 
 ### ODBC
@@ -138,3 +136,9 @@ Description = SQLite3 ODBC Driver
 Driver      = /opt/homebrew/Cellar/sqliteodbc/0.99991/lib/libsqlite3odbc.dylib
 ```
 4. Test configuration by running `odbcinst -q -d -n SQLite3`. If the path is printed out correctly, then you are all set.
+
+## Examples (in Python)
+1. Start a Python venv
+2. Enter into venv
+3. Inside python/ folder, run `maturin develop`.
+4. Inside python/examples/ folder, run the corresponding test using `python3 [file_name]`.

core/examples/duckdb.rs

@@ -19,7 +19,7 @@ async fn main() {
     // Opening in ReadOnly mode allows multiple reader processes to access
     // the database at the same time
     let duckdb_pool = Arc::new(
-        DuckDbConnectionPool::new_file("examples/duckdb_example.db", &AccessMode::ReadOnly)
+        DuckDbConnectionPool::new_file("core/examples/duckdb_example.db", &AccessMode::ReadOnly)
             .expect("unable to create DuckDB connection pool"),
     );
 

core/examples/sqlite.rs

@@ -20,7 +20,7 @@ async fn main() {
     // - arg3: Connection timeout duration
     let sqlite_pool = Arc::new(
         SqliteConnectionPoolFactory::new(
-            "examples/sqlite_example.db",
+            "core/examples/sqlite_example.db",
             Mode::File,
             Duration::from_millis(5000),
         )

core/src/sql/db_connection_pool/postgrespool.rs

@@ -1,4 +1,4 @@
-use std::{collections::HashMap, future::Future, path::PathBuf, str::FromStr, sync::Arc};
+use std::{collections::HashMap, path::PathBuf, str::FromStr, sync::Arc};
 
 use crate::{
     util::{self, ns_lookup::verify_ns_lookup_and_tcp_connect},