[GitHub] [iceberg] szehon-ho commented on pull request #6175: Hive: Add UGI to the key in CachedClientPool

[GitBox <gi...@apache.org>]

szehon-ho commented on PR #6175:
URL: https://github.com/apache/iceberg/pull/6175#issuecomment-1319877477

   I like that idea, to have a pluggable cache.  
   
   As I mentioned before with things like https://github.com/apache/iceberg/pull/5378 , we have users wanting to get different HMS clients based on different hive configs and unpleasantly surprised when they return same HMS client with old configs.  It is kind of a large burden for Iceberg to have to make cache key for every different possible configuration.
   
   Side note, we also tried to disable the cache before as well but it led to other errors:  https://github.com/apache/iceberg/issues/5371
   
   FYI @flyrain @RussellSpitzer , had discussed with them recently about this problem.


-- 
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: issues-unsubscribe@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org