adutra commented on code in PR #3658:
URL: https://github.com/apache/polaris/pull/3658#discussion_r2760954675
##########
runtime/service/src/intTest/java/org/apache/polaris/service/it/PolarisRestCatalogMinIOIT.java:
##########
@@ -88,4 +95,15 @@ protected StorageConfigInfo getStorageConfigInfo() {
return storageConfig.build();
}
+
+ @Override
+ protected Map<String, String> extraCatalogProperties(TestInfo testInfo) {
+ if
(testInfo.getTestMethod().orElseThrow().getName().equals("testRegisterTable")) {
+ // This test registers a table – operation that doesn't support access
delegation –
+ // then attempts to use the table's FileIO. This can only work if the
client has its
+ // own S3 credentials.
Review Comment:
It's a problem in the spec: the `registerTable` endpoint does not declare
the `data-access` header parameter:
https://github.com/apache/iceberg/blob/d0654440a4cd00feaff73c53cd3ef9fb787b92dc/open-api/rest-catalog-open-api.yaml#L883-L898
I will reach out to the Iceberg ML
--
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]
