Skip to content

[MINOR]: doc(iceberg): add warehouse document for REST catalog backend for Iceberg - #9811

Merged
jerryshao merged 4 commits into
apache:mainfrom
FANNG1:rest_warehouse
Jan 28, 2026
Merged

[MINOR]: doc(iceberg): add warehouse document for REST catalog backend for Iceberg#9811
jerryshao merged 4 commits into
apache:mainfrom
FANNG1:rest_warehouse

Conversation

@FANNG1

@FANNG1 FANNG1 commented Jan 27, 2026

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

add warehouse document for REST catalog backend for Iceberg

Why are the changes needed?

warehouse for REST catalog backend is different for other catalog backend

Does this PR introduce any user-facing change?

no

How was this patch tested?

just document

#### REST catalog backend

For the REST catalog backend, `warehouse` identifies the catalog in the Iceberg REST spec. In the Gravitino Iceberg REST server, `warehouse` maps to the catalog name. The default value is empty, which corresponds to the default catalog.

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.

Can you please give an example?

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.

updated

Copilot AI left a comment

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copilot AI left a comment

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.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread docs/lakehouse-iceberg-catalog.md Outdated
Comment thread docs/lakehouse-iceberg-catalog.md Outdated
@jerryshao jerryshao added the branch-1.1 Automatically cherry-pick commit to branch-1.1 label Jan 28, 2026
@jerryshao
jerryshao merged commit daaf252 into apache:main Jan 28, 2026
23 checks passed
github-actions Bot pushed a commit that referenced this pull request Jan 28, 2026
…d for Iceberg (#9811)

### What changes were proposed in this pull request?

add warehouse document for REST catalog backend for Iceberg

### Why are the changes needed?

warehouse for REST catalog backend is different for other catalog
backend

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
just document
FANNG1 pushed a commit to FANNG1/gravitino that referenced this pull request Mar 6, 2026
…d for Iceberg (apache#9811)

### What changes were proposed in this pull request?

add warehouse document for REST catalog backend for Iceberg

### Why are the changes needed?

warehouse for REST catalog backend is different for other catalog
backend

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
just document
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Jun 8, 2026
…d for Iceberg (apache#9811)

### What changes were proposed in this pull request?

add warehouse document for REST catalog backend for Iceberg

### Why are the changes needed?

warehouse for REST catalog backend is different for other catalog
backend

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
just document
danhuawang pushed a commit to danhuawang/gravitino that referenced this pull request Jun 9, 2026
…d for Iceberg (apache#9811)

### What changes were proposed in this pull request?

add warehouse document for REST catalog backend for Iceberg

### Why are the changes needed?

warehouse for REST catalog backend is different for other catalog
backend

### Does this PR introduce _any_ user-facing change?
no

### How was this patch tested?
just document
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch-1.1 Automatically cherry-pick commit to branch-1.1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants