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 any 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]
