jackye1995 commented on code in PR #9695:
URL: https://github.com/apache/iceberg/pull/9695#discussion_r1623316674
##########
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:
Review Comment:
@rdblue I think you suggested 422 for this behavior, but since we now use
422 for response too big case, I feel 421 is pretty suitable. This is too some
extent a "misdirected request" since the client is supposed to call preplan
first (based on information like server capability). Let me know if you agree
if 421 is suitable.
--
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
