eric-maynard commented on code in PR #1170:
URL: https://github.com/apache/polaris/pull/1170#discussion_r1994293707
##########
polaris-core/src/main/java/org/apache/polaris/core/config/BehaviorChangeConfiguration.java:
##########
@@ -37,6 +37,7 @@ protected BehaviorChangeConfiguration(
super(key, description, defaultValue, catalogConfig);
}
+ @BehaviorChangeConfigurationSince("1.0.0")
public static final BehaviorChangeConfiguration<Boolean>
VALIDATE_VIEW_LOCATION_OVERLAP =
Review Comment:
To answer your question more directly, in 1.2.0 this flag will expire, and
the linter will prompt the person attempting to do the release to take 1 of 3
actions:
* Remove the flag (the flag is not useful and its default becomes the only
code path)
* Promote the flag to a feature flag (the flag is deemed useful enough to
keep around under a new name)
* Disable the linter for this flag for an "extension" (we're still not sure
if the flag is useful)
I'm open to any and all improvements to this process, different ways of
surfacing this check, or better ways to file an "extension"
--
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]
