Petals CDK

Memory leak because the JBIAcceptorManager.acceptorPool is not freed

Details

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

    JBIAcceptorManager manages it acceptor thread pool using a ThreadPoolExecutor. ThreadPoolExecutor is designed to dispatch tasks to threads. The task are finished in the time.
    But JBIAcceptorManager does not used tasks. It uses the ThreadFactory to create it's own thread, with its own method 'run'.

    So this thread pool MUST be refactored to be a real thread pool (for example, as an array of threads)

    Show
    JBIAcceptorManager manages it acceptor thread pool using a ThreadPoolExecutor. ThreadPoolExecutor is designed to dispatch tasks to threads. The task are finished in the time. But JBIAcceptorManager does not used tasks. It uses the ThreadFactory to create it's own thread, with its own method 'run'. So this thread pool MUST be refactored to be a real thread pool (for example, as an array of threads)
  • Environment:
    -
  1. PETALSCDK-83.patch
    (9 kB)
    Christophe DENEUX
    Thu, 22 Mar 2012 - 18:00:05 +0100

Issue Links

Activity

Hide
Christophe DENEUX added a comment - Thu, 22 Mar 2012 - 18:00:05 +0100

A patch implementing the new thread pool. Caution, a new parameter is needed to drive the stop of acceptor thread (time to wait before to force the stop)

Show
Christophe DENEUX added a comment - Thu, 22 Mar 2012 - 18:00:05 +0100 A patch implementing the new thread pool. Caution, a new parameter is needed to drive the stop of acceptor thread (time to wait before to force the stop)
Hide
Christophe DENEUX added a comment - Fri, 23 Mar 2012 - 16:53:34 +0100

Fixed in branch petals-enterprise-3.1.x

Show
Christophe DENEUX added a comment - Fri, 23 Mar 2012 - 16:53:34 +0100 Fixed in branch petals-enterprise-3.1.x
Hide
Christophe DENEUX added a comment - Fri, 23 Mar 2012 - 16:53:42 +0100

To merge in trunk

Show
Christophe DENEUX added a comment - Fri, 23 Mar 2012 - 16:53:42 +0100 To merge in trunk
Hide
Christophe DENEUX added a comment - Mon, 26 Mar 2012 - 12:18:13 +0200

Merged in trunk

Show
Christophe DENEUX added a comment - Mon, 26 Mar 2012 - 12:18:13 +0200 Merged in trunk

People

Dates

  • Created:
    Thu, 22 Mar 2012 - 17:47:40 +0100
    Updated:
    Mon, 26 Mar 2012 - 12:18:13 +0200
    Resolved:
    Mon, 26 Mar 2012 - 12:18:13 +0200