danielcweeks commented on code in PR #18080:
URL: https://github.com/apache/iceberg/pull/18080#discussion_r4499163881
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -5847,6 +5872,11 @@ components:
description: The storage provider for which the request is to be
signed. The provider should correspond to
the scheme used for a storage native URI. For example `s3` for AWS
S3 paths. For backwards compatibility,
if this is not specified, the provider is assumed to be `s3`.
+ plan-id:
+ type: string
+ description: >
+ Optional identifier of the server side scan plan, supplied for
audit and correlation. A server
+ MUST NOT fail a request because the referenced plan is unknown,
expired, or cancelled.
Review Comment:
Agree with @singhpk234 here. We shouldn't be directing implementations that
they must not fail requests (in general).
--
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]
