flyrain commented on code in PR #2784:
URL: https://github.com/apache/polaris/pull/2784#discussion_r2421253302
##########
polaris-core/src/main/java/org/apache/polaris/core/config/FeatureConfiguration.java:
##########
@@ -429,4 +429,13 @@ public static void enforceFeatureEnabledOrThrow(
"When true, enables finer grained update table privileges which
are passed to the authorizer for update table operations")
.defaultValue(true)
.buildFeatureConfiguration();
+
+ public static final FeatureConfiguration<Boolean>
ALLOW_FEDERATED_CATALOGS_CREDENTIAL_VENDING =
Review Comment:
It's reasonable to introduce the config now. It may not be necessary for
long-term. We might think of a way to remove these configs. They makes system
unnecessary complex. Can we mark it to be removed at certain versions, e.g.,
version 1.3/1.4? and file an issue to remove these in case we forgot it.
--
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]
