Petals CDK

Memory-leak because utility singletons are not freed

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 5.1.3
  • Fix Version/s: 5.1.4
  • Component/s: runtime
  • Security Level: Public
  • Description:
    Hide

    The CDK uses several singleton as utility:

    • org.ow2.petals.component.framework.util.UtilFactory
    • org.ow2.petals.component.framework.util.ExchangeUtil
    • org.ow2.petals.component.framework.util.WSDLUtil
    • org.ow2.petals.component.framework.util.XSLTUtil
    • org.ow2.petals.component.framework.util.SourceUtil

    As these singletons are not freed, a new instance is created each time a component is installed because each component instance has its own classloader

    Show
    The CDK uses several singleton as utility:
    • org.ow2.petals.component.framework.util.UtilFactory
    • org.ow2.petals.component.framework.util.ExchangeUtil
    • org.ow2.petals.component.framework.util.WSDLUtil
    • org.ow2.petals.component.framework.util.XSLTUtil
    • org.ow2.petals.component.framework.util.SourceUtil
    As these singletons are not freed, a new instance is created each time a component is installed because each component instance has its own classloader
  • Environment:
    -

Issue Links

Depends
 

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Christophe DENEUX added a comment - Thu, 8 Mar 2012 - 11:33:46 +0100

Merge in branch petals-entreprise-3.1.x

Show
Christophe DENEUX added a comment - Thu, 8 Mar 2012 - 11:33:46 +0100 Merge in branch petals-entreprise-3.1.x
Hide
Permalink
Christophe DENEUX added a comment - Thu, 8 Mar 2012 - 11:35:49 +0100

To merge in trunk.

Is it needed to have singleton for utility classes. I think it is better and simpler to have only classes with static methods.

Show
Christophe DENEUX added a comment - Thu, 8 Mar 2012 - 11:35:49 +0100 To merge in trunk. Is it needed to have singleton for utility classes. I think it is better and simpler to have only classes with static methods.
Hide
Permalink
Christophe DENEUX added a comment - Fri, 9 Mar 2012 - 12:56:36 +0100 - edited

This issue has not to be merged in trunk since utility singletons were remplaced by static utility methods

Show
Christophe DENEUX added a comment - Fri, 9 Mar 2012 - 12:56:36 +0100 - edited This issue has not to be merged in trunk since utility singletons were remplaced by static utility methods
Hide
Permalink
Christophe DENEUX added a comment - Tue, 27 Mar 2012 - 16:58:10 +0200

CAUTION: new commit (svn#31906) in branch to prevent NPE when restarting/reinstalling the component, the UtilFactory must be freed by destructor

Show
Christophe DENEUX added a comment - Tue, 27 Mar 2012 - 16:58:10 +0200 CAUTION: new commit (svn#31906) in branch to prevent NPE when restarting/reinstalling the component, the UtilFactory must be freed by destructor

People

Dates

  • Created:
    Thu, 8 Mar 2012 - 10:43:16 +0100
    Updated:
    Thu, 29 Mar 2012 - 10:59:09 +0200
    Resolved:
    Thu, 29 Mar 2012 - 10:59:09 +0200
Atlassian JIRA (v4.1.2#531) | Report a problem | Request a feature | Contact administrators
Powered by a free Atlassian JIRA open source license for Petals ESB. Try JIRA - bug tracking software for your team.