okumin commented on PR #5887: URL: https://github.com/apache/hive/pull/5887#issuecomment-2994195117
> I guess we should merge it before the No Auth PR? I think so. Regression tests will unblock many feature developments, including caching. > btw, i've noticed that we are missing a few APIs in HMSCatalogAdapter: There is a small trick here. When `/v1/config` returns an empty list as available endpoints, [the REST client does not use the three endpoints](https://github.com/apache/iceberg/blob/apache-iceberg-1.9.1/core/src/main/java/org/apache/iceberg/rest/RESTSessionCatalog.java#L114-L132). That's why our implementation passes all the tests. I will update `/v1/config` and implement the three when we implement view-related endpoints. -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
