geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sangjin Lee (JIRA)" <>
Subject [jira] Updated: (GERONIMO-3720) the thread pool should be properly shut down in AHC sample app
Date Wed, 26 Dec 2007 21:21:43 GMT


Sangjin Lee updated GERONIMO-3720:

    Attachment: 3720.patch

a suggested fix

> the thread pool should be properly shut down in AHC sample app
> --------------------------------------------------------------
>                 Key: GERONIMO-3720
>                 URL:
>             Project: Geronimo
>          Issue Type: Bug
>      Security Level: public(Regular issues) 
>          Components: sample apps
>    Affects Versions: 1.x
>            Reporter: Sangjin Lee
>            Priority: Minor
>         Attachments: 3720.patch
> There was a recent API change with AsyncHttpClient, and the lifecycle management of the
thread pool now belongs to the caller (see GERONIMO-3707).  As a result, the thread pool needs
to be shut down properly in the servlet's destroy method.
> In addition, the current sample creates a bounded thread pool (using Executors.newFixedThreadPool()).
 If you do a stress test on this servlet, this doesn't work well, as it causes an unnecessary
performance penalty.  Mina recommends using an unbounded thread pool, and this thread pool
is usually very well regulated in practice, so there is no need to use a bounded thread pool
here.  See

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

View raw message