[ 
https://issues.apache.org/jira/browse/MYFACES-3549?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13278760#comment-13278760
 ] 

Jesús Pérez Alcaide (ISBAN) commented on MYFACES-3549:
------------------------------------------------------

Leonardo, what I mean is that the real problem was introduced in the solution 
to MYFACES-3262, where a strong reference was used to hold the methods in 
_PropertyDescriptorHolder. If you use a SoftReference, like 
java.beans.PropertyDescriptor uses internally, the whole problem is solved.

So, if you change that, there is no problem with ClassLoaders, there is no need 
to do initialization nor cleanup code, and there is no memory leak.
                
> Alternative solution to bugs MYFACES-3262 and MYFACES-3510
> ----------------------------------------------------------
>
>                 Key: MYFACES-3549
>                 URL: https://issues.apache.org/jira/browse/MYFACES-3549
>             Project: MyFaces Core
>          Issue Type: Improvement
>    Affects Versions: 2.1.7
>            Reporter: Jesús Pérez Alcaide (ISBAN)
>         Attachments: AAA_patch_PropertyDescriptor_MyFaces217.jar, 
> MyFaces3262_PropertyDescriptorHolder.patch, 
> MyFaces3510_AbstractFacesInitializer.patch, 
> MyFaces3510_ComponentAttributesMap.patch
>
>
> I would like to propose an alternative solution to bugs MYFACES-3262 and 
> MYFACES-3510, which consists in rollback the changes made in MYFACES-3510 and 
> modify the solution given in MYFACES-3262 by adding a soft reference to the 
> methods held in the class _PropertyDescriptorHolder.
> This alternative solution fixes MYFACES-3262 and avoids the performance loss 
> caused by MYFACES-3510.
> More information in this email:
> http://markmail.org/message/xh66wlte6luqn63d

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira


Reply via email to