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
Christophe DENEUX made changes - Thu, 8 Mar 2012 - 10:43:48 +0100
Field Original Value New Value
Link This issue blocks PETALSDISTRIB-2 [ PETALSDISTRIB-2 ]
Christophe DENEUX made changes - Thu, 8 Mar 2012 - 11:31:59 +0100
Status New [ 10000 ] Open [ 10002 ]
Priority Major [ 3 ]
Assignee Mathieu Carrolle [ mcarrolle ] Christophe DENEUX [ cdeneux ]
Christophe DENEUX made changes - Thu, 8 Mar 2012 - 11:32:01 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
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
Christophe DENEUX made changes - Thu, 8 Mar 2012 - 11:33:46 +0100
Status In Progress [ 10003 ] Resolved [ 10004 ]
Fix Version/s 5.1.4 [ 10328 ]
Resolution Fixed [ 1 ]
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.
Christophe DENEUX made changes - Thu, 8 Mar 2012 - 11:35:49 +0100
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Fri, 9 Mar 2012 - 12:44:54 +0100
Status Open [ 10002 ] In Progress [ 10003 ]
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
Christophe DENEUX made changes - Fri, 9 Mar 2012 - 12:56:36 +0100
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
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
Christophe DENEUX made changes - Tue, 27 Mar 2012 - 17:04:27 +0200
Status Resolved [ 10004 ] Open [ 10002 ]
Resolution Fixed [ 1 ]
Christophe DENEUX made changes - Thu, 29 Mar 2012 - 10:58:25 +0200
Comment [ Reopen to merge the complementary commit (see previous comment) ]
Christophe DENEUX made changes - Thu, 29 Mar 2012 - 10:59:01 +0200
Status Open [ 10002 ] In Progress [ 10003 ]
Christophe DENEUX made changes - Thu, 29 Mar 2012 - 10:59:09 +0200
Status In Progress [ 10003 ] Resolved [ 10004 ]
Resolution Fixed [ 1 ]
Transition Status Change Time Execution Times Last Executer Last Execution Date
New New Open Open
48m 43s
1
Christophe DENEUX
Thu, 8 Mar 2012 - 11:31:59 +0100
Open Open In Progress In Progress
2s
1
Christophe DENEUX
Thu, 8 Mar 2012 - 11:32:01 +0100
In Progress In Progress Resolved Resolved
1m 45s
1
Christophe DENEUX
Thu, 8 Mar 2012 - 11:33:46 +0100
Resolved Resolved Open Open
2m 3s
1
Christophe DENEUX
Thu, 8 Mar 2012 - 11:35:49 +0100
Open Open In Progress In Progress
1d 1h 9m
1
Christophe DENEUX
Fri, 9 Mar 2012 - 12:44:54 +0100
In Progress In Progress Resolved Resolved
11m 42s
1
Christophe DENEUX
Fri, 9 Mar 2012 - 12:56:36 +0100
Resolved Resolved Open Open
18d 3h 7m
1
Christophe DENEUX
Tue, 27 Mar 2012 - 17:04:27 +0200
Open Open In Progress In Progress
1d 17h 54m
1
Christophe DENEUX
Thu, 29 Mar 2012 - 10:59:01 +0200
In Progress In Progress Resolved Resolved
8s
1
Christophe DENEUX
Thu, 29 Mar 2012 - 10:59:09 +0200



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.