rdblue commented on code in PR #9695:
URL: https://github.com/apache/iceberg/pull/9695#discussion_r1692075370
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -537,6 +537,113 @@ paths:
5XX:
$ref: '#/components/responses/ServerErrorResponse'
+ /v1/{prefix}/namespaces/{namespace}/tables/{table}/preplan:
Review Comment:
I think I did mean either 421 or 422 and not 521. The return codes that
redirect the client to the preplan endpoint.
I think there's a fair point that if we only have one capability, clients
can call either endpoint first. I'd definitely _recommend_ calling `plan`
first, but that's up to the client and, more importantly, because we don't
_need_ to require a certain behavior we should not require a certain behavior.
Specs should be flexible and leave room for implementations to be smart.
--
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
