Seonggon Namgung

Results 31 comments of Seonggon Namgung

@BsoBird , thanks for pointing out the typo. I fixed it.

@okumin, I have added an Appendix section to the design document, which summarizes the dependencies between MetaStoreClients as well as which classes in the PR codebase provide specific features compared...

@deniskuzZ, Thank you for looking over the PR. I think the suggested class names make sense, and I'll update the PR accordingly. Regarding `HiveMetaStoreClientWithTmpTable`, my initial thought was that we...

@okumin , @deniskuzZ , @zratkai, thank you for your review. I've updated the branch to address your feedback and improve code quality. The main changes are summarized as follows: -...

> I found Hive 4.0.0 has around 20 deprecated methods. Are we allowed to discontinue them before [HIVE-27473](https://issues.apache.org/jira/browse/HIVE-27473)? I think removing the deprecated methods would be helpful for simplifying the...

> 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...

@okumin, I understand now. Thank you for the explanation. I’ve also started a discussion on the mailing list. It would be great if you could take a look and help...

@okumin, I have rebased the PR. @deniskuzZ, As far as I know, there are no blockers we need to consider at this point.

@okumin, @dengzhhu653, Thank you for your exhaustive reviews. Based on your comments, I summarize the remaining tasks as follows: 1. Interface normalization issue 2. FilterHook-related issues 3. Thread-safety issue (particularly...