Skip to content

Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md - #17937

Closed
talatuyarer wants to merge 1 commit into
apache:mainfrom
talatuyarer:docs-rest-client-properties
Closed

Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md#17937
talatuyarer wants to merge 1 commit into
apache:mainfrom
talatuyarer:docs-rest-client-properties

Conversation

@talatuyarer

Copy link
Copy Markdown
Contributor

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.

@github-actions github-actions Bot added the docs label Sep 3, 2026
@talatuyarer talatuyarer changed the title Docs: Document update for Rest Catalog's HTTP client properties in catalog-properties.md Docs, Core: Document update for Rest Catalog's HTTP client properties in catalog-properties.md Sep 3, 2026
| `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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a dedicated RESTCatalogProperties. Wouldn't these new configs fit better there?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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: #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

@talatuyarer talatuyarer closed this Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants