We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
main (development)
Currently table_exists only returns true on 200:
iceberg-python/pyiceberg/catalog/rest.py
Line 793 in 20b7b53
According to the spec, it should return true on 204 (and probably 200 as well): https://github.com/apache/iceberg/blob/ea916c1700ef37377d3e8bba1197b3fe32d1a248/open-api/rest-catalog-open-api.yaml#L820-L829
The text was updated successfully, but these errors were encountered:
@c-thiel Great catch! I agree, let's accept both 200 and 204. Are you interested in fixing this?
Sorry, something went wrong.
Sure - PR is coming. I'll have some more tricky great catches tomorrow ;)
Successfully merging a pull request may close this issue.
Apache Iceberg version
main (development)
Please describe the bug 馃悶
Currently table_exists only returns true on 200:
iceberg-python/pyiceberg/catalog/rest.py
Line 793 in 20b7b53
According to the spec, it should return true on 204 (and probably 200 as well):
https://github.com/apache/iceberg/blob/ea916c1700ef37377d3e8bba1197b3fe32d1a248/open-api/rest-catalog-open-api.yaml#L820-L829
The text was updated successfully, but these errors were encountered: