huaxingao commented on code in PR #14196:
URL: https://github.com/apache/iceberg/pull/14196#discussion_r2422279925
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -1373,12 +1441,17 @@ paths:
TableToUpdateDoesNotExist:
$ref: '#/components/examples/NoSuchTableError'
409:
- description:
- Conflict - CommitFailedException, one or more requirements failed.
The client may retry.
+ description: |
+ Conflict - Multiple conflict scenarios possible:
+ - CommitFailedException, one or more requirements failed. The
client may retry.
Review Comment:
Thanks for your suggestion! I prefer to stay with the original message,
because adding `with a different Idempotency-Key` makes it sound like
idempotency support is required for 409. Since idempotency is optional, I think
keeping this generic is better.
--
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]
