tmater commented on issue #2168: URL: https://github.com/apache/polaris/issues/2168#issuecomment-3127982628
I just want to confirm that I understand the issue correctly. Currently, in some places, we're using `CommitFailedException`. While it generally describes the situation and returns the correct 409 response, it's not the best fit since it's an Iceberg-specific exception. In some of these cases, the conflict actually originates from Polaris. We should review where `CommitFailedException` is used and consider replacing it with a more appropriate and specific `CommitConflictException` when the conflict comes from Polaris. -- 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...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
