You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by "talatuyarer (via GitHub)" <gi...@apache.org> on 2026/09/03 05:02:33 UTC
[PR] Docs: Document update for Rest Catalog's HTTP client properties in catalog-properties.md [iceberg]
talatuyarer opened a new pull request, #17937:
URL: https://github.com/apache/iceberg/pull/17937
Adds a new "HTTP client properties" subsection under "REST catalog properties" in `docs/docs/catalog-properties.md`, documenting all 11 `rest.client.*` keys defined in `HTTPClient.java`, which previously had no documentation anywhere in the docs.
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md [iceberg]
Posted by "talatuyarer (via GitHub)" <gi...@apache.org>.
talatuyarer commented on code in PR #17937:
URL: https://github.com/apache/iceberg/pull/17937#discussion_r3930618683
##########
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:
Thanks for the suggestion, @gaborkaszab. Your feedback prompted me to consider why we don't have a dedicated REST Catalog documentation page. I have addressed this in a new PR: https://github.com/apache/iceberg/pull/17940, where I've created dedicated pages for the REST Protocol and the REST Catalog.
I will be appricated if you can review and share your thoughts
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md [iceberg]
Posted by "talatuyarer (via GitHub)" <gi...@apache.org>.
talatuyarer closed pull request #17937: Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md
URL: https://github.com/apache/iceberg/pull/17937
--
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org
Re: [PR] Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md [iceberg]
Posted by "gaborkaszab (via GitHub)" <gi...@apache.org>.
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: issues-unsubscribe@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org