Some cleanup

Author: matthewmturner

.github/workflows/test.yml

@@ -272,7 +272,7 @@ jobs:
         run: |
           cargo test --manifest-path crates/datafusion-udfs-wasm/Cargo.toml
   test-auth:
-    name: Extension / Auth-Basic-Bearer
+    name: Extension / Auth
     runs-on: ubuntu-latest
     strategy:
       matrix:
@@ -293,6 +293,7 @@ jobs:
           key: ${{ runner.os }}-cargo-target-${{ hashFiles('Cargo.lock') }}
       - name: Setup Rust Toolchain
         uses: ./.github/actions/setup-rust
+      # Right now basic and bearer auth can not be used at the same time.  So we serve an instance with one, then kill it, then serve an instance with another.  An alternative, perhaps simpler option, would be configuring the ports used - but this would come at the cost of a more annoying local dev experience as you have to remember to keep updating the ports.
       - name: Start FlightSQL Server with Basic Auth
         run: |
           cargo r --features=flightsql -- serve-flightsql --config data/configs/flightsql_basic.toml & echo $! > server.pid

crates/datafusion-app/Cargo.toml

@@ -3,6 +3,8 @@ edition = "2021"
 name = "datafusion-app"
 version = "0.1.0"
 
+# Leaving Hudi and Iceberg around in case traction on those picks up.  It would be better if I just remove them from main and create PRs with the diff - maybe ill get around to that.
+
 [dependencies]
 arrow-flight = { version = "54.1.0", features = [
   "flight-sql-experimental",

crates/datafusion-app/src/extensions/mod.rs

@@ -62,8 +62,8 @@ pub fn enabled_extensions() -> Vec<Arc<dyn Extension>> {
         Arc::new(deltalake::DeltaLakeExtension::new()),
         // #[cfg(feature = "hudi")]
         // Arc::new(hudi::HudiExtension::new()),
-        #[cfg(feature = "iceberg")]
-        Arc::new(iceberg::IcebergExtension::new()),
+        // #[cfg(feature = "iceberg")]
+        // Arc::new(iceberg::IcebergExtension::new()),
         #[cfg(feature = "huggingface")]
         Arc::new(huggingface::HuggingFaceExtension::new()),
     ]