You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "David Jencks (JIRA)" <de...@myfaces.apache.org> on 2007/03/11 23:59:09 UTC

[jira] Created: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

FactoryFinder not thread safe, and has a memory leak
----------------------------------------------------

                 Key: MYFACES-1558
                 URL: https://issues.apache.org/jira/browse/MYFACES-1558
             Project: MyFaces Core
          Issue Type: Bug
          Components: General
    Affects Versions: 1.2.0-SNAPSHOT
            Reporter: David Jencks


FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

Posted by "David Jencks (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/MYFACES-1558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12479973 ] 

David Jencks commented on MYFACES-1558:
---------------------------------------

Thanks for looking into this.  I have been working with what you get from checking out current12, and the code started with generics in it.  I'm not familiar with your version numbering, and assumed this corresponded to version 1.2.0-SNAPSHOT.  What MyFaces version should I be calling it?

> FactoryFinder not thread safe, and has a memory leak
> ----------------------------------------------------
>
>                 Key: MYFACES-1558
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1558
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: General
>    Affects Versions: 1.2.0-SNAPSHOT
>            Reporter: David Jencks
>         Assigned To: Dennis Byrne
>         Attachments: MYFACES-1558.patch
>
>
> FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

Posted by "Dennis Byrne (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/MYFACES-1558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12479971 ] 

Dennis Byrne commented on MYFACES-1558:
---------------------------------------

Hi David,

Thanks for you participation lately.  Can you do us a favor?  The JSF 1.1 spec requires java 1.3 compatibility, so we can use generics.  The JSF 1.2 spec allows java 1.5.  We're working on a 1.2 implementation over in another branch:

svn checkout http://svn.apache.org/repos/asf/myfaces/current12 myfaces

> FactoryFinder not thread safe, and has a memory leak
> ----------------------------------------------------
>
>                 Key: MYFACES-1558
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1558
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: General
>    Affects Versions: 1.2.0-SNAPSHOT
>            Reporter: David Jencks
>         Attachments: MYFACES-1558.patch
>
>
> FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

Posted by "Dennis Byrne (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/MYFACES-1558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12479972 ] 

Dennis Byrne commented on MYFACES-1558:
---------------------------------------

we can use generics -> we can not use generics

> FactoryFinder not thread safe, and has a memory leak
> ----------------------------------------------------
>
>                 Key: MYFACES-1558
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1558
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: General
>    Affects Versions: 1.2.0-SNAPSHOT
>            Reporter: David Jencks
>         Assigned To: Dennis Byrne
>         Attachments: MYFACES-1558.patch
>
>
> FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


[jira] Commented: (MYFACES-1558) FactoryFinder not thread safe, and has a memory leak

Posted by "Dennis Byrne (JIRA)" <de...@myfaces.apache.org>.
    [ https://issues.apache.org/jira/browse/MYFACES-1558?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12479976 ] 

Dennis Byrne commented on MYFACES-1558:
---------------------------------------

OK, if you are working w/ myfaces/current12 then you are fine, it's the JSF 1.2 branch.  I can't look at this tonight but it'll happen soon.  

If you have time, a non-generics version of the patch (for myfaces/current, the JSF 1.1 maintenance mode branch) would be appreciated.

Thanks

> FactoryFinder not thread safe, and has a memory leak
> ----------------------------------------------------
>
>                 Key: MYFACES-1558
>                 URL: https://issues.apache.org/jira/browse/MYFACES-1558
>             Project: MyFaces Core
>          Issue Type: Bug
>          Components: General
>    Affects Versions: 1.2.0-SNAPSHOT
>            Reporter: David Jencks
>         Assigned To: Dennis Byrne
>         Attachments: MYFACES-1558.patch
>
>
> FactoryFinder has insufficient synchronization to be thread safe, and has a memory leak in _registeredFactoryNames.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.