dimas-b commented on code in PR #4052:
URL: https://github.com/apache/polaris/pull/4052#discussion_r3075000837
##########
runtime/service/src/main/java/org/apache/polaris/service/catalog/iceberg/IcebergCatalogHandler.java:
##########
@@ -812,6 +816,13 @@ public Optional<LoadTableResponse> loadTable(
// when data-access is specified but access delegation grants are not
found.
Table table = baseCatalog.loadTable(tableIdentifier);
+ // Capture tableId from remote catalog response for S3 Tables ARN
construction
+ Optional<String> capturedTableId = capturedConfigHolder().getTableId();
+ List<String> resourceArns = List.of();
+ if (capturedTableId.isPresent()) {
+ resourceArns = List.of(constructS3TablesArn(capturedTableId.get()));
Review Comment:
@aritragster : just to sync up: I tend to prefer completing #3699 first... I
hope it does not cause too much delay on this PR. Please let us know if this is
timeline is problematic.
#3699 may not provide all the features needed by this PR, but at least I
hope we can perform the main refactoring in #3699 and later add remaining
features for S3 tables in this PR.
--
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]
