commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ben McCann (JIRA)" <>
Subject [jira] [Commented] (BCEL-186) Release 6.0
Date Tue, 30 Dec 2014 15:21:13 GMT


Ben McCann commented on BCEL-186:

Hi Emmanuel. I opened the bug as a place to attach patches for release blockers. I'm afraid
you saw the bug before I had a chance to attach the patch so that wasn't clear. Perhaps we
can reopen and possibly rename this issue? In any case, I hope you'll take a look at the attached
patch which resolves the issue brought up in the last RC vote. Thanks again for your help.

> Release 6.0
> -----------
>                 Key: BCEL-186
>                 URL:
>             Project: Commons BCEL
>          Issue Type: Task
>    Affects Versions: 6.0
>            Reporter: Ben McCann
>            Priority: Blocker
>         Attachments: revert-ConstantUtf8-cache.diff
> I'd love to see 6.0 released.
> Here was the last vote:
> Konstantin Kolinko reported some issues with caching that blocked the release:
> bq. I noted new methods that introduced caching into classfile.ConstantUtf8 class. BCEL_DONT_CACHE
should at least have static setter, or a static setter for a cache instance / a factory class.
RELEASE-NOTES.txt describes this change as [BCEL-163] Incorporate patch file from Findbugs
which does not say much about this change. Actually this change introduced caching for ConstantUtf8
and ObjectType instances.
> Emmanuel Bourg did some investigation and testing and reported back:
> bq. If nobody object I'll remove this cache, the impact on the performance is too important
to enable it by default, and the static state smells like a quick and dirty implementation.
This feature could return as a pluggable cache if someone wants to provide a patch.
> garydgregory suggested adding a flush method to clear the cache in ObjectType

This message was sent by Atlassian JIRA

View raw message