difin commented on code in PR #6474:
URL: https://github.com/apache/hive/pull/6474#discussion_r3561236582
##########
iceberg/iceberg-catalog/src/main/java/org/apache/iceberg/hive/IcebergCatalogProperties.java:
##########
@@ -104,6 +105,19 @@ public static String catalogPropertyConfigKey(String
catalogName, String catalog
return String.format("%s%s.%s", CATALOG_CONFIG_PREFIX, catalogName,
catalogProperty);
}
+ /**
+ * Returns true when the catalog is configured to request REST vended
storage credentials via
+ * {@link #REST_ACCESS_DELEGATION_HEADER_PROPERTY}.
+ */
+ public static boolean requestsVendedCredentials(String catalogName,
Configuration conf) {
Review Comment:
I think it is better to leave it in `IcebergCatalogProperties` because this
this catalog configuration method and it is also called from
`TestRestAccessDelegationMode` which lives in `iceberg-catalog` module.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
