HonahX commented on code in PR #2784:
URL: https://github.com/apache/polaris/pull/2784#discussion_r2421393285
##########
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:
I agree this should be the default behavior. But I think it may be still
good to preserve the ability to toggle the behavior until the new feature
stabilized. And yeah, may be several versions afterwards, we could remove this
flag completely and make this a fundamental feature : )
--
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]
