Re: RFR: 8161203: ResourceBundle.getBundle performance regression

2016-07-24 Thread Masayoshi Okutsu
Hi Peter, The change (ResourceBundle part) looks very good to me. There's a simple workaround for the problem -- call ResourceBundle.clearCache(ClassLoader). So I don't know how serious the problem is, though. I still like this change. Yes, the comment in ReferenceTest should be removed. Th

Re: RFR: 8161203: ResourceBundle.getBundle performance regression

2016-07-24 Thread Alan Bateman
On 22/07/2016 14:13, Peter Levart wrote: : The changes are very straightforward. They preserve the general structure of the logic. I renamed the CacheKey nested class to LoadSession as it now only functions as a mutable object passed around the methods while executing the getBundle() process