myfaces-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jesús Pérez Alcaide (ISBAN) (JIRA) <...@myfaces.apache.org>
Subject [jira] [Commented] (MYFACES-3549) Alternative solution to bugs MYFACES-3262 and MYFACES-3510
Date Fri, 18 May 2012 12:18:08 GMT

    [ 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

       

Mime
View raw message