szlta opened a new pull request, #5536: URL: https://github.com/apache/iceberg/pull/5536
As discussed in #5507 SetSnapshotOperation currently doesn't commit the operation if the metadata is unchanged. Since this is used in a transaction, it can result in failure of committing the whole transaction as the `hasLastOpCommited` was not advanced. I propose we commit such empty operations, as the overall transaction commit will result in a no-op anyway in these cases. cc: @kbendick @amogh-jahagirdar -- 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]
