Gabriel39 commented on code in PR #67711:
URL: https://github.com/apache/doris/pull/67711#discussion_r3966353618
##########
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/plans/commands/ExecuteActionCommand.java:
##########
@@ -144,6 +144,23 @@ public Optional<Expression> getWhereCondition() {
return whereCondition;
}
+ private ResultSet executeAuthenticated(ExecuteAction action, ExternalTable
table) throws Exception {
+ try {
+ // Iceberg tables retain filesystem configuration, not the
caller's UGI, so loading the table and
+ // committing its metadata must stay within one catalog
authentication scope.
+ return table.getCatalog().getExecutionAuthenticator().execute(()
-> {
Review Comment:
Fixed in e921629bb4. Action creation now captures the metadata ops and
authenticator atomically under the catalog reset monitor. Writable table
acquisition is fenced by the captured ops; if the catalog generation changes
before mutation, the command rebuilds the action and retries once under the new
authenticator. Commit failures are not retryable. The command test now
deterministically switches from authenticator A to B before writable
acquisition and verifies that generation A never loads or commits while
generation B commits exactly once.
--
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]
