Skip to content

add iceberg check namespace exists api#65

Open
kenandominic wants to merge 1 commit intounitycatalog:mainfrom
kenandominic:main
Open

add iceberg check namespace exists api#65
kenandominic wants to merge 1 commit intounitycatalog:mainfrom
kenandominic:main

Conversation

@kenandominic
Copy link

PR Checklist

  • A description of the changes is added to the description of this PR.
  • If there is a related issue, make sure it is linked to this PR.
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added or modified a feature, documentation in docs is updated

Description of changes

  • adds support for the HEAD /v1/namespaces/{namespace} endpoint
  • modifies the HEAD /v1/namespaces/{namespace}/tables/{table} endpoint to respond with an http response code of 204 as per spec
  • modifies the GET /v1/namespaces and GET /v1/namespaces/{namespace} endpoints to respond with an http response code of 400 if the namespace is invalid; contains more than two parts
  • fixes the ordering of the "actual" and "expected" arguments for asserting http response codes in IcebergRestCatalogTest.java

@kenandominic
Copy link
Author

hey, I'd like to work on adding support for the iceberg api endpoints that are listed here if that's okay. I've started with one of the more simpler endpoints just to get a feel of the codebase.

Sign up for free to 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.

1 participant