Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Core: Remove namespace/table/view HEAD endpoints from defaults #12351

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

nastra
Copy link
Contributor

@nastra nastra commented Feb 20, 2025

No description provided.

@nastra nastra force-pushed the rest-backwards-comp-head-requests branch from b8e7f24 to fb94311 Compare February 20, 2025 13:58
@nastra nastra changed the title Core: Remove namespace/table/view HEAD requests from defaults Core: Remove namespace/table/view HEAD endpoints from defaults Feb 20, 2025
@@ -111,8 +111,6 @@ paths:

- GET /v1/{prefix}/namespaces/{namespace}

- HEAD /v1/{prefix}/namespaces/{namespace}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

note that this was added after #11756 and needs to be removed again, since we can't actually modify/update the default endpoints in the RESTSessionCatalog that were initially defined by #10929

@nastra nastra added this to the Iceberg 1.8.1 milestone Feb 20, 2025
Copy link
Contributor

@ebyhr ebyhr left a comment

Choose a reason for hiding this comment

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

I verified this PR fixes Trino test failures.

@nastra nastra merged commit 4958663 into apache:main Feb 21, 2025
43 checks passed
@nastra nastra deleted the rest-backwards-comp-head-requests branch February 21, 2025 08:42
nastra added a commit to nastra/iceberg that referenced this pull request Feb 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants