yuqi1129 commented on PR #11783: URL: https://github.com/apache/gravitino/pull/11783#issuecomment-5068369768
@puchengy Would you like to push forward with this one as discussed offline? > we should make use of [lance-format/lance-namespace-impls@main/java/lance-namespace-hive2](https://github.com/lance-format/lance-namespace-impls/tree/main/java/lance-namespace-hive2?rgh-link-date=2026-06-24T04%3A42%3A33Z) instead of rebuilding lance hive table operations classes? hive2 depends on hadoop2 and is conflicting with gravitino common's hadoop3 dependency Since Gravitino has already defined a common backend API like `NamespaceWrapper`, I believe it's okay to rebuild Lance hive table operations classes. For Hive2, Gravitino depends on Hive2 and Hive3 at the same time, this should not be a big problem. > How do we want to support hive 3/ 4, do we want a different profile or do we want to use hive2 for hive3/4 client? Some basic reasearch shows that hive2 usage here will be compatible with hive3/4. The API should be generally compatible with most API for Hive2/Hive3/Hive4, we can resolve them case by case if we did plan to support those versions. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
