Fokko commented on issue #1018: URL: https://github.com/apache/iceberg-python/issues/1018#issuecomment-2307278079
Thanks everyone for bringing this up. Long term, I think we should leverage the endpoint discovery to see if the server provides the capability of table-exists: https://lists.apache.org/thread/8h86382omdx9cmvc15m2bf361p5rz4rk Since the table-exists is in there from the beginning, I think we should follow the spec, and assume that: - **204**: Table exists. - **404**: Table doesn't exist. - Raise an exception for the rest, similar to other operations. The HEAD operation does not require the REST Catalog to fetch the metadata, which will consume less resources on the server, but probably also return much faster for the client. I think need to update the Java implementation to use the HEAD request, this will also uncover the issue on the Java side. For validation, hopefully, we can check this properly using https://github.com/apache/iceberg/pull/10908 once that is in. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
