eric-maynard commented on PR #433: URL: https://github.com/apache/polaris/pull/433#issuecomment-2501529018
Hi @snazy; I agree that the EntityCache is potentially a problem. I think it can be optimized pretty easily with a [Weigher](https://www.javadoc.io/doc/com.github.ben-manes.caffeine/caffeine/2.6.0/com/github/benmanes/caffeine/cache/Weigher.html). Although the metastore caching is disabled by default, do you view this as a blocker by itself? Indeed originally in older versions of this PR I had the metadata skip the EntityCache, but @dennishuo convinced me that in many cases it's quite useful to have the in-memory caching. -- 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]
