[hibernate-issues] [Hibernate-JIRA] Commented: (HHH-5927) Performance risk: Suboptimal synchronization in org.hibernate.engine.query.QueryPlanCache.getHQLQueryPlan

[ https://hibernate.onjira.com/browse/HHH-5927?page=com.atlassian.jira.plug... ] Eric Dalquist commented on HHH-5927: ------------------------------------ So the problem with MapMaker is you're completely giving up the max-size configuration and relying solely on the GC to cleanup, the big danger there is soft-reference only caches can turn into serious performance problems in low-memory high-demand situations as data is constantly removed from the cache, increasing CPU demands. The dual-map approach addresses this fairly well by keeping more frequently accessed data available no matter what the GC state is. As you noted, the equality check isn't an issue since you're never comparing the values in the cache. I believe in my example you can remove the synchronized keywords as well with the one problem that there could be a cache miss while an entry is being evicted from the strong cache and moved into the weak cache. If we can assume that this case would be rare it wouldn't be a big problem as the plan would be re-generated and placed back into the strong cache, usurping the existing entry. So we would have the very small chance for some extra CPU work with the benefit of being lock-free Also in your gist you set concurrency to 1, are you really only expecting 1 thread to update the table at a time? -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira