jackye1995 commented on code in PR #9695:
URL: https://github.com/apache/iceberg/pull/9695#discussion_r1623316510
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3804,6 +4110,41 @@ components:
}
}
+ # Note that this is a representative example response for use as a
shorthand in the spec.
+ # The fields `message` and `type` as indicated here are not presently
prescriptive.
+ MisdirectedRequestResponse:
+ description:
+ Misdirected Request. The request was directed to a server that is not
able to produce a response.
+ content:
+ application/json:
+ schema:
+ $ref: '#/components/schemas/IcebergErrorResponse'
+ example: {
+ "error": {
+ "message": "Fail to plan table scan: must perform preplan before
planning a table scan",
+ "type": "PreplanTableRequiredException",
+ "code": 421
+ }
+ }
+
+ # Note that this is a representative example response for use as a
shorthand in the spec.
+ # The fields `message` and `type` as indicated here are not presently
prescriptive.
+ UnprocessableContentResponse:
Review Comment:
I think @rdblue you suggested using 413 for payload too big, but based on
the HTTP code, it looks like 413 is for Request payload too big, but what we
want here is response payload too big. I think it can be covered by 422, so I
changed to that with an example
--
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...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
