Shohei Okumiya
Shohei Okumiya
I'm reimplementing it here https://issues.apache.org/jira/browse/HIVE-28411
I added the recommended change. I did not add a test case because we're replacing the existing ones with the more comprehensive test suite in https://issues.apache.org/jira/browse/HIVE-29019. I am feeling the...
Merged. @deniskuzZ @henrib Thanks for your review comments!
The content in Google Doc is perfectly aligned with my memory. I'm still reading the proposed changes. I'd appreciate it if the latest dependencies between MetaStoreClients somewhere were written somewhere
I found Hive 4.0.0 has around 20 deprecated methods. Are we allowed to discontinue them before HIVE-27473? https://github.com/apache/hive/blob/rel/release-4.0.0/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
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...
@ngsg > when you mentioned “existing integrations,” were you referring to integration tests, or to external integrations such as third-party catalogs? I know Glue Data Catalog depends on the unmerged...
I happened to find Apache Ranger also relies on IMetaStoreClient. I believe HIVE-27473 does not impact the feature, but I am leaving a comment nonetheless. https://github.com/apache/ranger/blob/master/hive-agent/src/main/java/org/apache/ranger/authorization/hive/authorizer/RangerHiveAuthorizer.java
@deniskuzZ Hi, thanks for asking. As [everyone supports the current option](https://lists.apache.org/thread/xcmrnz6p5c7b27wbzv379t9dzfymof7z), we can proceed with this. No one has raised specific concerns at this point. @ngsg Could you rebase this...
My review progress is visible here. https://gist.github.com/okumin/add5b97c3b6674820cf1aad433856e9f @ngsg @deniskuzZ @dengzhhu653 While reviewing the pull request, I found MetaStoreFilterHook is used in 40+ places, and I guess we no longer need...