Set default database when connecting to Materialize (#4)

Adds a default `materialize` database to the default connection string.
MaterializeInc · Feb 4, 2020 · a7771f6 · a7771f6
2 parents 53aef03 + 2b2fd43
commit a7771f6
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 4 deletions.
diff --git a/images/materialize-metabase.png b/images/materialize-metabase.png
diff --git a/resources/metabase-plugin.yaml b/resources/metabase-plugin.yaml
@@ -14,10 +14,13 @@ driver:
   connection-properties:
     - merge:
         - host
-        - default: localhost
+        - default: materialized
     - merge:
         - port
         - default: 6875
+    - merge:
+        - dbname
+        - default: materialize
     - merge:
         - user
         - default: default

diff --git a/src/metabase/driver/materialize.clj b/src/metabase/driver/materialize.clj
@@ -15,17 +15,17 @@
 ; ;;; +----------------------------------------------------------------------------------------------------------------+
 
 (defn- make-subname [host port db]
-  (str "//" host ":" port "/"))
+  (str "//" host ":" port "/" db))
 
 (defn materialize
   "Create a Clojure JDBC database specification for Materialize."
   [{:keys [host port db]
-    :or   {host "localhost", port 6875, db ""}
+    :or   {host "localhost", port 6875, db "materialize"}
     :as   opts}]
   (merge
    {:classname                     "io.materialize.Driver"
     :subprotocol                   "materialize"
-    :subname                       (make-subname host (or port 6875) db)}
+    :subname                       (make-subname host port db)}
    (dissoc opts :host :port :db)))