excalibur-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Markus Wiederkehr (JIRA)" <...@excalibur.apache.org>
Subject [jira] Commented: (EXLBR-20) org.apache.avalon.excalibur.thread.impl.DefaultThreadPool is not disposed properly.
Date Mon, 15 Sep 2008 12:03:44 GMT

    [ https://issues.apache.org/jira/browse/EXLBR-20?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12631004#action_12631004

Markus Wiederkehr commented on EXLBR-20:

I think a new bug was introduced by fixing this one and this issue should be re-opened.

A ThreadGroup cannot be destroyed if it still contains threads. ThreadGroup throws an IllegalThreadStateException
if the group is not empty.

More specifically the DefaultThreadPool code from version 2.1 looks like this:
  public void dispose()

m_pool is a BasicThreadPool which in turn has an underlying pool also called m_pool. Now assume
that underlying pool is a SoftResourceLimitingPool from excalibur-pool-impl-2.1.jar.

The dispose() method of SoftResourceLimitingPool (declared in DefaultPool) removed all Poolable
elements from its ready pool m_ready. But it does not do anything about the active elements
in m_active.

As a consequence the thread group may not be empty and an IllegalThreadStateException is thrown.

See also JAMES-767 for an issue where this actually happens.

> org.apache.avalon.excalibur.thread.impl.DefaultThreadPool is not disposed properly.
> -----------------------------------------------------------------------------------
>                 Key: EXLBR-20
>                 URL: https://issues.apache.org/jira/browse/EXLBR-20
>             Project: Excalibur Components
>          Issue Type: Bug
>          Components: Thread
>            Reporter: Pawel Stawicki
>            Assignee: Shash Chatterjee
>            Priority: Minor
> DefaultThreadPool extends ThreadGroup. When it is created and added to parent ThreadGroup,
ThreadGroup holds reference to it. When dispose is called, parent ThreadGroup still has reference
to it and garbage collector can not collect it. It causes memory leak.
> This can be solved by calling destroy() in dispose method.

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

To unsubscribe, e-mail: dev-unsubscribe@excalibur.apache.org
For additional commands, e-mail: dev-help@excalibur.apache.org

View raw message