gaborkaszab commented on code in PR #17937:
URL: https://github.com/apache/iceberg/pull/17937#discussion_r3923132860
##########
docs/docs/catalog-properties.md:
##########
@@ -56,6 +56,24 @@ The following properties configure the behavior of the REST
catalog client.
| `namespace-separator` | `%1F` | The separator
character used for namespace levels when communicating with the REST server.
|
| `scan-planning-mode` | `CLIENT` | Controls where
scan planning is performed. Supported values: `CLIENT` (client-side planning),
`SERVER` (server-side planning). Can be overridden per-table by the server in
LoadTableResponse. |
+### HTTP client properties
Review Comment:
There is a dedicated `RESTCatalogProperties`. Wouldn't these new configs fit
better there?
--
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]
