dimas-b commented on code in PR #3259:
URL: https://github.com/apache/polaris/pull/3259#discussion_r2615373971
##########
polaris-core/src/main/java/org/apache/polaris/core/persistence/bootstrap/SchemaOptions.java:
##########
@@ -25,4 +25,6 @@
@PolarisImmutable
public interface SchemaOptions {
Optional<Integer> schemaVersion();
+
+ Optional<String> schemaFile();
Review Comment:
I'm not sure about this change. `schemaFile()` was present at some point,
but got removed in #2762.
Since this PR does not have a real call paths leading to it, would you mind
deferring this change? I imagine (if it has to be done) the Admin tool will be
affected too. WDYT?
--
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]
