dimas-b commented on code in PR #1959:
URL: https://github.com/apache/polaris/pull/1959#discussion_r2180866062
##########
polaris-core/src/main/java/org/apache/polaris/core/config/PolarisConfiguration.java:
##########
@@ -41,6 +43,7 @@ public abstract class PolarisConfiguration<T> {
public final String key;
public final String description;
+ public final Set<String> legacyKeys;
Review Comment:
Thx! To clarify: old config entries should keep working until we drop
support for them (e.g. in 2.0). Let's add a WARN log message when we have to
fall back to the "legacy" property name so that users are notified in runtime
and are more aware of the need to migrating to the new `key`.
--
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
