Re: RFR: 8161203: ResourceBundle.getBundle performance regression

2016-07-22 Thread Peter Levart
On 07/22/2016 10:46 AM, Peter Levart wrote: Hi Masayoshi, Here's a preview of work to migrate ResourceBundle caching to using ClassLoaderValue utility: http://cr.openjdk.java.net/~plevart/jdk9-dev/ResourceBundle.cleanup/webrev.04/ The changes are very straightforward. They preserve the gene

Re: RFR: 8161203: ResourceBundle.getBundle performance regression

2016-07-22 Thread Peter Levart
Hi Masayoshi, Thinking of how the ResourceBundle caching is implemented, I think it has a serious flaw. The cache is currently the following: private static final ConcurrentMap cacheList ...where the CacheKey is an object which contains WeakReference(s) to the caller's ClassLoader and Module