RFR: 8214712: Archive Attributes$Name.KNOWN_NAMES
Peter Levart
peter.levart at gmail.com
Fri Mar 15 16:07:27 UTC 2019
Hi Claes,
If you have observed memory churn allocating Name objects when reading
jar files, then perhaps we could do something to the logic of Attributes
class so that lazily allocated Name(s) would get interned too?
As a separate change of course. This looks good as is.
Regards, Peter
On 3/15/19 2:45 PM, Claes Redestad wrote:
> Hi,
>
> On 2019-03-14 18:20, Claes Redestad wrote:
>>
>> On 2019-03-14 18:13, Alan Bateman wrote:
>>>
>>> For the current webrev then I'm concerned it is brings back legacy
>>> attributes. The concept of "installed optional packages" was removed
>>> in Java SE 9, as was the ability for JAR packaged applets to trigger
>>> downloading of optional packages. I don't think the later was ever
>>> implemented in the JDK so surprising that we are finding JAR files
>>> with those attributes now. If we can prune that down then I think
>>> the changes will be okay.
>>
>> Ok. I stumbled on some new test issues in SA with this patch, so I'll
>> need to pause this one for a while anyhow.
>
> I have a solution to the heap dump issues out for review[1], so I've
> cleaned up this patch, verified the failing tests pass locally and am
> running both through tier1-3:
>
> http://cr.openjdk.java.net/~redestad/8214712/open.01/
>
> Thanks!
>
> /Claes
>
> [1]
> http://mail.openjdk.java.net/pipermail/serviceability-dev/2019-March/027455.html
More information about the core-libs-dev
mailing list