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

Bug Fix: Expose tableExists endpoint on RESTCatalogAdapter #11678

Closed
wants to merge 1 commit into from

Conversation

sungwy
Copy link
Contributor

@sungwy sungwy commented Nov 29, 2024

expose HEAD request endpoint at /v1/{prefix}/namespaces/{namespace}/tables/{table} path to be used for tableExists method calls as specified by The Open API Spec

head:
tags:
- Catalog API
summary: Check if a table exists
operationId: tableExists
description:
Check if a table exists within a given namespace. The response does not contain a body.
responses:
204:
description: Success, no content
400:
$ref: '#/components/responses/BadRequestErrorResponse'
401:
$ref: '#/components/responses/UnauthorizedResponse'
403:
$ref: '#/components/responses/ForbiddenResponse'

@ebyhr
Copy link
Contributor

ebyhr commented Dec 1, 2024

This PR looks a duplicate of #10999. The relevant issue is #10993

@Fokko
Copy link
Contributor

Fokko commented Dec 2, 2024

Thanks @ebyhr for pointing that out, and sorry for leaving that PR lingering. Thanks to @sungwy and @ebyhr for fixing this 🙌 I will close this one now since it is a duplicate.

@Fokko Fokko closed this Dec 2, 2024
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.

3 participants