-
Notifications
You must be signed in to change notification settings - Fork 209
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
Add table_exists
method to the Catalog
#507
Comments
could be similar to the mkdirs api? With |
@jayceslesar I would keep it simple by just returning a bool. |
@kevinjqliu is this supported ?
|
@djouallah No that's not expected. If it doesn't supported the specialized |
If it's open to work, I can take a shot at it! |
@MehulBatra That would be great 👍 The main this is that we need to implement this method: iceberg-python/pyiceberg/catalog/__init__.py Lines 387 to 396 in 88a26e1
|
PR: #678 |
Hi everyone, I think we've had the implementation. The only "issue" is that this feature hasn't been released yet. I'll add this one to 0.7.0 milestone. @MehulBatra Thanks for the PR btw. It is good to have more tests for this feature |
Thanks @HonahX Let me close it for now 👍 |
Feature Request / Improvement
Prolonging the discussion in #406
I think having this method on the
Catalog
would be a great addition. On top of this, we can also leverage the table-exists implementation of the REST catalog by overriding this method in the REST catalog.The text was updated successfully, but these errors were encountered: