RussellSpitzer commented on code in PR #395:
URL: https://github.com/apache/polaris/pull/395#discussion_r1813252657


##########
polaris-core/src/main/java/org/apache/polaris/core/PolarisConfiguration.java:
##########
@@ -154,6 +154,14 @@ public static <T> Builder<T> builder() {
           .defaultValue(false)
           .build();
 
+  public static final PolarisConfiguration<Boolean> 
ALLOW_EXTERNAL_CATALOG_CREDENTIAL_VENDING =

Review Comment:
   I would agree with @eric-maynard and @sfc-gh-dhuo . Ideally I would probably 
like to be able to set this on a per catalog basis but having blanket coverage 
is nice too. In the future I would like to have a more flexible way defining 
policies like this.



-- 
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]

Reply via email to