[GitHub] [iceberg] nastra commented on pull request #7310: Hive: Clean up expired metastore clients

["nastra (via GitHub)" <gi...@apache.org>]

nastra commented on PR #7310:
URL: https://github.com/apache/iceberg/pull/7310#issuecomment-1506476128

   > Caffeine cache will not always call removelistener by default, so an extra scheduler is required to invoke removelistener.
   
   From how I understand how Caffeine does this is that it performs the cleanup whenever the cache is being accessed or modified.
   
   > If expireAfter, expireAfterWrite, or expireAfterAccess is requested then entries may be evicted on each cache modification, on occasional cache accesses, or on calls to Cache.cleanUp. A scheduler(Scheduler) may be specified to provide prompt removal of expired entries rather than waiting until activity triggers the periodic maintenance. Expired entries may be counted by Cache.estimatedSize(), but will never be visible to read or write operations.
   
   That means the removal listener will be called. The question for this PR should rather be how **critical** it is that the removal is performed immediately vs waiting until activity triggers the periodic maintenance.


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