roryqi commented on code in PR #10767:
URL: https://github.com/apache/gravitino/pull/10767#discussion_r3076976142
##########
iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/dispatcher/IcebergTableHookDispatcher.java:
##########
@@ -75,7 +72,22 @@ public LoadTableResponse updateTable(
IcebergRequestContext context,
TableIdentifier tableIdentifier,
UpdateTableRequest updateTableRequest) {
- return dispatcher.updateTable(context, tableIdentifier,
updateTableRequest);
+ LoadTableResponse response =
+ dispatcher.updateTable(context, tableIdentifier, updateTableRequest);
+ // Import the table and set ownership if the entity does not yet exist,
+ // e.g. when committing a staged table create.
+ try {
Review Comment:
UpdateTable isn't only used for committing a staged table create. It can be
used for update table properties.
It is not reasonable to set the table table owner if we only update the
table properties.
--
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]
