adutra commented on code in PR #4812:
URL: https://github.com/apache/polaris/pull/4812#discussion_r3431127205


##########
runtime/admin/src/main/resources/application.properties:
##########
@@ -31,7 +31,12 @@ quarkus.container-image.registry=docker.io
 quarkus.container-image.group=apache
 quarkus.container-image.name=polaris-admin-tool
 quarkus.container-image.additional-tags=latest
-quarkus.datasource.db-kind=postgresql
+
+# Named datasources for relational persistence backends.
+# Polaris ships with many built-in datasources; by using named datasources, 
users can configure
+# at runtime which datasources to use for relational persistence backends.
+quarkus.datasource.postgresql.db-kind=postgresql

Review Comment:
   The admin module does not ship H2, at least for now.
   
   We may want to include H2 later. There are pros and cons:
   
   - con: running the admin tool on an ephemeral database is meaningless
   - pro: running the admin tool on an ephemeral database allows users to play 
with the different commands without having to configure a database.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to