snazy opened a new pull request, #8413: URL: https://github.com/apache/iceberg/pull/8413
`NessieTableOperations.doCommit()` wrongly assumes that _every_ exception thrown during a Nessie commit operation is a failure that requires the deletion of the newly written table-metadata. However, for exceptions like `j.l.InterruptedException` or HTTP timeout/error it is unclear whether the Nessie commit went through (or will go through), so deleting the table-metadata is wrong here. This change updates the logic to only delete the table-metadata file if a `NessieReferenceConflictException` happened, when it is clear that the commit really failed. -- 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]
