Merge pull request #4 from J0hnG4lt/feat/add-github-action-and-use-deps-edn

feat: add github action and use deps.end

Author: J0hnG4lt

.github/workflows/build.yaml

@@ -0,0 +1,73 @@
+name: Build Metabase arrow-flight-sql Driver
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+  workflow_dispatch:
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - name: Clone Metabase repository
+      uses: actions/checkout@v4
+      with:
+        repository: metabase/metabase
+        path: metabase
+
+    - name: Checkout driver code
+      uses: actions/checkout@v4
+      with:
+        path: metabase/modules/drivers/arrow-flight-sql
+
+    - name: List driver files for debugging
+      run: |
+        echo "Listing contents of metabase/modules/drivers/arrow-flight-sql:"
+        ls -R metabase/modules/drivers/
+        ls -R metabase/modules/drivers/arrow-flight-sql
+
+    - name: Register arrow-flight-sql in deps.edn
+      working-directory: ./metabase
+      run: |
+        cp modules/drivers/arrow-flight-sql/ci/deps.edn modules/drivers/deps.edn
+        echo "--- Verifying modules/drivers/deps.edn content ---"
+        cat modules/drivers/deps.edn
+        echo "----------------------------------------------------"
+
+
+    - name: Set up JDK 21
+      uses: actions/setup-java@v4
+      with:
+        distribution: 'temurin'
+        java-version: '21'
+
+    - name: Set up Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: 'lts/*'
+
+    - name: Install Yarn 1.x
+      run: |
+        npm install -g yarn@1
+        yarn --version
+
+    - name: Set up Clojure
+      uses: DeLaGuardo/setup-clojure@12.3
+      with:
+        cli: latest
+
+    - name: Build driver
+      working-directory: ./metabase
+      run: |
+        chmod +x ./bin/build-driver.sh
+        ./bin/build-driver.sh arrow-flight-sql
+
+    - name: Upload driver artifact
+      uses: actions/upload-artifact@v4
+      if: github.event_name == 'push' && github.ref == 'refs/heads/main'
+      with:
+        name: metabase-arrow-flight-sql-driver
+        path: metabase/resources/modules/arrow-flight-sql.metabase-driver.jar
+        if-no-files-found: error

ci/deps.edn

@@ -0,0 +1,27 @@
+{
+ ;; we have to copy over the Maven repos used for the Athena and Redshift drivers to here as well -- this is
+ ;; apparently done on purpose, as a security thing. See
+ ;; https://ask.clojure.org/index.php/10726/deps-manifest-dependencies-respect-repos-dependent-project
+ :mvn/repos
+ {"metabase-maven-downloads" {:url "https://s3.amazonaws.com/metabase-maven-downloads"}
+  "redshift" {:url "https://s3.amazonaws.com/redshift-maven-repository/release"}}
+
+ :deps
+ {metabase/athena             {:local/root "athena"}
+  metabase/bigquery-cloud-sdk {:local/root "bigquery-cloud-sdk"}
+  metabase/clickhouse         {:local/root "clickhouse"}
+  metabase/databricks         {:local/root "databricks"}
+  metabase/druid              {:local/root "druid"}
+  metabase/druid-jdbc         {:local/root "druid-jdbc"}
+  metabase/hive-like          {:local/root "hive-like"}
+  metabase/mongo              {:local/root "mongo"}
+  metabase/oracle             {:local/root "oracle"}
+  metabase/presto-jdbc        {:local/root "presto-jdbc"}
+  metabase/redshift           {:local/root "redshift"}
+  metabase/snowflake          {:local/root "snowflake"}
+  metabase/sparksql           {:local/root "sparksql"}
+  metabase/sqlite             {:local/root "sqlite"}
+  metabase/sqlserver          {:local/root "sqlserver"}
+  metabase/starburst          {:local/root "starburst"}
+  metabase/vertica            {:local/root "vertica"}
+  metabase/arrow-flight-sql   {:local/root "arrow-flight-sql"}}}

deps.edn

@@ -0,0 +1,6 @@
+{:paths ["src" "resources"]
+
+ :deps {org.clojure/clojure {:mvn/version "1.11.1"}
+        org.apache.arrow/flight-sql-jdbc-driver {:mvn/version "18.2.0"}}
+
+}

project.clj

@@ -5,4 +5,4 @@
             :url "https://www.apache.org/licenses/LICENSE-2.0"}
   :dependencies [[org.clojure/clojure "1.11.1"]
                  [org.apache.arrow/flight-sql-jdbc-driver "18.2.0"]]
-  :repl-options {:init-ns metabase.driver.flightsql})
+  :repl-options {:init-ns metabase.driver.arrow-flight-sql})

resources/metabase-plugin.yaml

@@ -36,6 +36,6 @@ driver:
 
 init:
   - step: load-namespace
-    namespace:  metabase.driver.flightsql
+    namespace:  metabase.driver.arrow-flight-sql
   - step: register-jdbc-driver
     class: org.apache.arrow.driver.jdbc.ArrowFlightJdbcDriver

src/metabase/driver/flightsql.clj src/metabase/driver/arrow_flight_sql.cljsrc/metabase/driver/flightsql.clj renamed to src/metabase/driver/arrow_flight_sql.clj

@@ -1,4 +1,4 @@
-(ns metabase.driver.flightsql
+(ns metabase.driver.arrow-flight-sql
   "Arrow Flight SQL Driver for Metabase.
 
   This driver uses the Apache Arrow Flight SQL JDBC driver.
@@ -329,7 +329,7 @@
   (.setTimestamp stmt idx (Timestamp/valueOf dt)))
 
 (defmethod sql-jdbc.execute/set-parameter
-  [:arrow‑flight‑sql OffsetDateTime]
+  [:arrow-flight-sql OffsetDateTime]
   [_driver ^PreparedStatement stmt ^Integer idx ^OffsetDateTime odt]
   ;; convert OffsetDateTime → Timestamp at the same instant
   (.setTimestamp stmt