eric-maynard commented on code in PR #1672:
URL: https://github.com/apache/polaris/pull/1672#discussion_r2107778202
##########
polaris-core/src/main/java/org/apache/polaris/core/config/PolarisConfiguration.java:
##########
@@ -162,13 +162,8 @@ public Builder<T> catalogConfig(String catalogConfig) {
return this;
}
- /**
- * Used to support backwards compatability before there were reserved
properties. Usage of this
- * method should be removed over time.
- */
- @Deprecated
Review Comment:
That was definitely my intent when I made it -- to prohibit new usage and
eventually phase out old usage. I think there's a comment to this effect, and
the alerts are sort of "meant" to be annoying. We shouldn't have configs that
are unsafe, I think.
--
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]
