danielcweeks commented on code in PR #9523:
URL: https://github.com/apache/iceberg/pull/9523#discussion_r1459891880


##########
core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java:
##########
@@ -1022,11 +1022,17 @@ public void commitTransaction(SessionContext context, 
List<TableCommit> commits)
           UpdateTableRequest.create(commit.identifier(), 
commit.requirements(), commit.updates()));
     }
 
+    Map<String, String> props = properties();
+    if (null != context.credentials()) {
+      props = RESTUtil.merge(properties(), context.credentials());
+    }
+
+    AuthSession authSession = tableSession(props, session(context));

Review Comment:
   I think we should use the catalog session here instead of a table session.  
The table session is intended to be scoped to a particular table, but this is a 
cross-table operation and should be handled as a catalog level using the 
`catalogAuth`



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to