Shohei Okumiya

I suspended the review because there are some cases where `SessionMetaStoreClientProxy` is unlikely to be invoked. It would be great if we could discontinue `MetaStoreFilterHook`, or we probably have to...

Sorry, MetaStoreFilterHook is used as HiveMetaStoreAuthorizer or AuthorizationMetaStoreFilterHook. I was searching only for the standalone-metastore project. My bad

@ngsg @dengzhhu653 @deniskuzZ I reviewed half of the methods and commented on where I found issues. My impression is that the number of methods is huge, and it is not...

I checked all methods and commented on all places where I had questions

Thanks. As we are cutting 4.1 and we are moving to 5.0 next, I would say we can remove some deprecated methods.

The intention is admirable, but I am not 100% confident that this is the best approach because I need to know how GROUPING SETS works. It still needs time for...

@BsoBird Thanks. Could you give us a document or equivalent implementation of Trino?

@mayankkunwar I recently resolved another hang issue with additional integration tests. This change can be a reference for your work. https://github.com/apache/tez/commit/44c4f1ec9fc2758c4f8ae0850b871c93720663fd#diff-ad65a331fa51a07f3cc5301ca7df09c199e9730a6f889bc8b1859554ccfc0519

@barking-code Thanks for locating the issue and providing the fix. Could you update the output of the failing test cases? Please feel free to ask me if you have any...

@barking-code Do you have a chance to update this branch? You can also take a look at a new document on how to run or update test cases. https://hive.apache.org/development/qtest/