Skip to content
Snippets Groups Projects
Commit 0127bc2d authored by Cam Saül's avatar Cam Saül Committed by GitHub
Browse files

Merge pull request #2888 from metabase/tiniest-code-cleanup

Remove a couple lines that are no longer needed :yum:
parents ee5ba9ec 3ee4fae8
No related branches found
No related tags found
No related merge requests found
......@@ -92,8 +92,7 @@
{:pre [(map? db-details)]}
;; TODO: it's probably a good idea to put some more validation here and be really strict about what's in `db-details`
(case (:type db-details)
:h2 (dbspec/h2 (assoc db-details :naming {:keys s/lower-case
:fields s/upper-case}))
:h2 (dbspec/h2 db-details)
:mysql (dbspec/mysql (assoc db-details :db (:dbname db-details)))
:postgres (dbspec/postgres (assoc db-details :db (:dbname db-details)))))
......@@ -226,7 +225,7 @@
(assert (binding [*allow-potentailly-unsafe-connections* true]
(require 'metabase.driver)
((resolve 'metabase.driver/can-connect-with-details?) engine details))
"Unable to connect to Metabase DB.")
(format "Unable to connect to Metabase %s DB." (name engine)))
(log/info (str "Verify Database Connection ... ✅")))
(defn setup-db
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment