[GitHub] [iceberg] pvary commented on pull request #6499: AWS, Core, Hive: Fix `checkCommitStatus` when create table commit fails

[GitBox <gi...@apache.org>]

pvary commented on PR #6499:
URL: https://github.com/apache/iceberg/pull/6499#issuecomment-1366719073

   Did some cursory review and left a small comment.
   
   When the commitStatus is UNKNOWN then we sign the user that the code was not able to decide how could it fix the state, and expects the user to fix the table. In this case we keep everything and expect the user to cleanup the state.
   
   If we can identify that the issue is with the access rights, and we can identify and fix the table, then we might throw a different exception and fix the table instead.
   
   I am on PTO till January. If the PR is still open, please ping me to do a full review.
   
   Thanks, Peter 


-- 
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-unsubscribe@iceberg.apache.org

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


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