[
https://issues.apache.org/jira/browse/GROOVY-7977?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15622693#comment-15622693
]
John Wagenleitner commented on GROOVY-7977:
-------------------------------------------
Hi Daniel,
My initial reaction is that it would be good to keep the classes/interface
together in a separate {{org.apache.groovy.util.concurrentlinkedhashmap}}
package and not mix with potential future util classes. If at some point in
the future it would become better to move them up into {{o.a.g.util}} then it
could be done at that time. But initially keeping them cleanly separated seems
good to me, just my opinion.
> Refine the implementation of LRUCache
> -------------------------------------
>
> Key: GROOVY-7977
> URL: https://issues.apache.org/jira/browse/GROOVY-7977
> Project: Groovy
> Issue Type: Improvement
> Affects Versions: 2.4.7
> Reporter: Daniel.Sun
> Priority: Minor
> Attachments: text.html, text.html, text.html, text.html, text.html,
> text.html, text.html
>
>
> I found the LRUCache that Groovy is using is not efficient enough
> because it is based on Collections.synchronizedMap, the LRUCache is used by
> some important classes(e.g. ProxyGenerator, Closure), so it can impact the
> performance somehow.
> I propose to replace the implementation with
> Caffeine(https://github.com/ben-manes/caffeine), which is a high performance
> caching library for Java 8 and licensed under APL2.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
