dimas-b commented on issue #2168: URL: https://github.com/apache/polaris/issues/2168#issuecomment-3129961557
Yes, `CommitFailedException` is currently used by Polaris code to represent eventual 409 errors in the REST API. However, Polaris uses a lot of non-REST Iceberg code and `CommitFailedException` may potentially occur in other contexts, where it is not clear what the appropriate REST status code should be. IMHO, it is preferable to use `CommitConflictException`, which is owned by Polaris and is intended to clearly map to 409 at the REST level. -- 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
