Skip to content

REST: Add missing endpoint check for creating views - #3442

Open
ebyhr wants to merge 1 commit into
apache:mainfrom
ebyhr:ebi/create-view-endpoint
Open

REST: Add missing endpoint check for creating views#3442
ebyhr wants to merge 1 commit into
apache:mainfrom
ebyhr:ebi/create-view-endpoint

Conversation

@ebyhr

@ebyhrebyhr commented May 30, 2026

Copy link
Copy Markdown
Member

Rationale for this change

The endpoint check for a new view creation is missing.

Iceberg Java also validates it https://github.com/apache/iceberg/blob/be27af46df4316b41787a62382c5aa330a173c1e/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java#L1670-L1672

Are these changes tested?

No

Are there any user-facing changes?

No

@ebyhr
ebyhrforce-pushed the ebi/create-view-endpoint branch from a812a14 to af2746fCompareMay 30, 2026 05:23
@github-actions

Copy link
Copy Markdown

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.

@ebyhr

ebyhr commented Jul 1, 2026

Copy link
Copy Markdown
MemberAuthor

@Fokko@kevinjqliu@geruh could you review this PR when you have time?

@github-actions

Copy link
Copy Markdown

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that's incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@iceberg.apache.org list. Thank you for your contributions.

@ebyhr
ebyhrforce-pushed the ebi/create-view-endpoint branch from af2746f to 1bbf05dCompareAugust 2, 2026 02:33

@rambleraptorrambleraptor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Easy enough. Thanks for doing this!

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@ebyhr@rambleraptor@abnobdoss