ngsg commented on PR #5771: URL: https://github.com/apache/hive/pull/5771#issuecomment-2826396753
> Can we raise the proposal to the mailing list and tie-break the best approaches at this point? People in #4444 should be very interested. Then, we can invest our resources in reviewing this pull request. > > The Option 2 of my original document might make integration clean. [We will add a small change to Apache Hive](https://docs.google.com/document/d/1fFvB0DAXJvPYv27R8nLa3tOUT67-hY0owxBrDY-BxUA/edit?tab=t.0#heading=h.82t8ookld7kn), and third-party vendors [implement the minimal number of methods](https://github.com/okumin/aws-glue-data-catalog-client-for-apache-hive-metastore/commit/aee2cc5bc4e6c59a0cc1e9f7ad0f6f05f4d92691). However, this approach doesn't allow existing integrations to run as they are. > > If we can make Option 1 clean and the community people agree with it, I think many people would be happy. Additionally, the HiveMetaStoreClient family with complicated dependencies will be decomposed! Sure, it would be nice to make an intermediate design decision at this point. Before raising the proposal on the mailing list, I have a small question about your comment: when you mentioned “existing integrations,” were you referring to integration tests, or to external integrations such as third-party catalogs? I’d like to make sure I fully understand the impact you're referring to, especially in terms of potential behavioral changes. -- 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
