N-o-Z commented on PR #9789:
URL: https://github.com/apache/iceberg/pull/9789#issuecomment-1964153139

   > I believe the purpose was that `CatalogHandlers.commit(..)` should only be 
used through `CatalogHandlers.updateTable(...)`. Is there a particular issue 
that prevents you from going through `CatalogHandlers.updateTable(...)`?
   
   AFAIU updateTable recieves a single UpdateTableRequest, and is used in the 
REST context to handle an `UPDATE_TABLE` request.
   On the other hand the `COMMIT_TRANSACTION` request receives a list of 
UpdateTableRequest and preforms updates on all requests as an atomic operation. 
   I don't see how this can be achieved using updateTable


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