singhpk234 commented on code in PR #1686:
URL: https://github.com/apache/polaris/pull/1686#discussion_r2167263637
##########
persistence/relational-jdbc/src/main/java/org/apache/polaris/persistence/relational/jdbc/DatabaseType.java:
##########
@@ -44,6 +44,6 @@ public static DatabaseType fromDisplayName(String
displayName) {
}
public String getInitScriptResource() {
- return String.format("%s/schema-v1.sql", this.getDisplayName());
+ return String.format("%s/schema-v2.sql", this.getDisplayName());
Review Comment:
sounds good, i think may be taking version at the time of bootstrapping in
from the command line and then percolating all the way through could also be an
option ? this way the database type is still inferred using the the existing
mechanism
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
