caican00 commented on PR #5443: URL: https://github.com/apache/gravitino/pull/5443#issuecomment-2456183119
> Hi @caican00 will there be any potential UGI issue when we reuse the client instance? CC @yuqi1129 @jerqi to also take a look. When Impersonation is disable, we use a super user to access hms, reusing the clients are reasonable. When Impersonation is enabled , we obtain a HiveClientPool from the cache through a `Key` instance. If the ugi or user_name is the same, the `Key` instance should be the same and the connection in the HiveClientPool should be taken instead of recreating a new HiveClientPool. Now each request recreates a HiveClientPool instance, the HiveClientPool is meaningless. Is my understanding correct? If there is any problem, please help point out. Thank you very much cc @yuqi1129 -- 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]
