singhpk234 commented on code in PR #14867:
URL: https://github.com/apache/iceberg/pull/14867#discussion_r2842270070
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3468,6 +3468,9 @@ components:
## General Configurations
- `token`: Authorization bearer token to use for table requests if
OAuth2 security is enabled
+ - `scan-planning-mode`: Controls scan planning behavior for table
operations. Valid values:
+ - `client` (default): Clients MUST use client-side scan planning
+ - `server`: Clients MUST use server-side scan planning if the server
supports it, otherwise MUST fall back to client-side planning
Review Comment:
Thats fair, we send the `server` supports in the `/config` endpoint and then
the `server` in the loadTable endpoint, this was mostly there to avoid
discrepancy on the responses on both the endpoint.
I can remove this and keep this from spec and can contain in the
implementation `Precondition` wdyt ?
--
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]
