Shohei Okumiya

Can we decompose this PR into multiple PRs based on the sub-tasks? I understand that some rewrites require the real JDK 17, but others don't. https://issues.apache.org/jira/browse/HIVE-26473 I also wonder if...

@akshat0395 Thanks! I'm willing to merge this one, but I have two challenges. ### The number of changed files is huge This pull request changes 178 files. I manually counted...

> We have a data connector for bridging over different resources for this purpose, do we consider using this feature? https://issues.apache.org/jira/browse/HIVE-24396 If I understand correctly, it requires us to have...

> @okumin Could you please address the comment in [the original pull request](https://github.com/apache/hive/pull/1402)? The limitation with this approach is we don't get the features implemented in SessionHiveMetastoreClient (Example: handling of...

@ganeshashree Thanks! I think this point is still a bit controversial. > I vote for users to specify an implementation of IMetaStoreClient via HiveConf and use that in SessionHiveMetastoreClient instead...

I'm checking the `HiveMetaStoreClient` family and I found it is not trivial. `HiveMetaStoreClient`, `HiveMetaStoreClientWithLocalCache`, and `SessionHiveMetaStoreClient` are tightly coupled by dynamic dispatching throw APIs defined in not `IMetaStoreClient` but `HiveMetaStoreClient`....

Let me implement the alternative idea and ask the community to conclude that later

@deniskuzZ @wecharyu @dengzhhu653 @ganeshashree Sorry for the late response. I tried to implement the suggested option and summarize the current points. To be honest, I still prefer the original option...

Thanks. I sent an e-mail to the dev ML for visibility. https://lists.apache.org/thread/607swrj6pq4g6q052tyo4l304vb091m2

@dengzhhu653 Thanks for the meaningful insight! > 1. Can we just expose SessionHiveMetaStoreClient in the Hive class? > 2. This makes sense to me, the SessionHiveMetastoreClient acts as a wrapper...