pvary commented on code in PR #14867:
URL: https://github.com/apache/iceberg/pull/14867#discussion_r2840661047
##########
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:
This is a bit strange to me.
The server sends this value. Why would it send `server` if the server not
supports it?
--
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]
