dimas-b commented on code in PR #3658:
URL: https://github.com/apache/polaris/pull/3658#discussion_r2760827903


##########
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:
   Interesting... Is it possible that Polaris neglects to return the 
credentials in the registerTable response, or is it a problem in the IRC API 
spec?



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