danielcweeks commented on code in PR #14867:
URL: https://github.com/apache/iceberg/pull/14867#discussion_r2688243187
##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -3374,6 +3374,29 @@ 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. This property can be configured by:
+ - **Server**: Returned in `LoadTableResponse.config()` to advertise
server preference/requirement
+ - **Client**: Set in catalog properties to override server
configuration
Review Comment:
I think we need to remove anything in the REST spec referencing the client
configuration. This spec should be explicitly reserved for what the server
needs to implement.
How a client configures/defines it's behavior should not be covered in the
spec (it's an implementation detail).
--
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]
