vigneshio opened a new pull request, #5145: URL: https://github.com/apache/polaris/pull/5145
Per the Iceberg REST spec (including Polaris's own vendored `spec/iceberg-rest-catalog-open-api.yaml`), `CommitStateUnknownException` on commit paths is a 5xx (500/502/504). Polaris mapped it to HTTP 400 in `IcebergExceptionMapper`. The Iceberg's `ErrorHandlers` commit error handler maps 409 → `CommitFailedException` and 500 → `CommitStateUnknownException`; a 400 falls through to the default handler, so compliant clients reconstruct `BadRequestException`. A commit that may have been applied then looks like a rejected request, which invites unsafe retries (duplicate appends). - `IcebergExceptionMapper`: `CommitStateUnknownException` → 500 (the spec's plain-server case; 502/504 are gateway flavors). - Pinning test covering commit-state-unknown (500) plus neighboring mappings (409/400) so case reordering cannot silently regress. -- 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]
