[jboss-jira] [JBoss JIRA] Commented: (JBMESSAGING-1356) Add thread pool for client/server sessions

[Ovidiu Feodorov (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBMESSAGING-1356?page=comments#action_12417494 ] 
            
Ovidiu Feodorov commented on JBMESSAGING-1356:
----------------------------------------------

For some reason, I didn't get the e-mail notifications on your comments.

The patch is applied on https://svn.jboss.org/repos/messaging/branches/Branch_Experimental_JBMESSAGING_1356_2/

Please let me know in which point release will it be available.

> Add thread pool for client/server sessions
> ------------------------------------------
>
>                 Key: JBMESSAGING-1356
>                 URL: http://jira.jboss.com/jira/browse/JBMESSAGING-1356
>             Project: JBoss Messaging
>          Issue Type: Feature Request
>          Components: Messaging Core
>    Affects Versions: 1.4.0.SP3.CP02
>            Reporter: Jay Howell
>         Assigned To: Ovidiu Feodorov
>             Fix For: 1.4.0.SP3_CP03
>
>
> The threads named jbm-server-session* and jbm-client-session* should be gotten from a thread pool instead of just being created.  There's no way to cap the number of session threads. 
> The thread factory is defined in NamedThreadQueuedExecutor as...
> private class Factory implements ThreadFactory
> 	{
> 		public Thread newThread(Runnable command)
> 		{			
> 			return new Thread(jbmGroup, command, name);
> 		}
> 		
> 	}
> This thread factory should really have a thread pool implementation and not just a new thread implementation.  This class controls both the server threads and the client threads.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira