[jboss-jira] [JBoss JIRA] Commented: (JBREM-860) Eliminate delay in MicroSocketClientInvoker.getConnection()

[Ron Sigal (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBREM-860?page=comments#action_12390370 ] 
            
Ron Sigal commented on JBREM-860:
---------------------------------

Tim Fox has supplied a fix to MicroSocketClientInvoker that uses semaphores.  

The fix has been applied to branch 2.2.2-GA_CP for EAP 4.3.  It should also be applied to branches 2.2 for JBAS 5.0.0.Beta3 and 2.x.

Test:  org.jboss.test.remoting.transport.socket.socketpool;

   org.jboss.test.remoting.transport.socket.socketpool.SocketClientPoolWithSemaphoreTestCase

The test passes in Windows and linux.  Also, the JBossMessaging test suite works with the fix in place.

> Eliminate delay in MicroSocketClientInvoker.getConnection()
> -----------------------------------------------------------
>
>                 Key: JBREM-860
>                 URL: http://jira.jboss.com/jira/browse/JBREM-860
>             Project: JBoss Remoting
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 2.2.2.SP2, 2.2.2.GA_CP01, 2.4.0.Beta1 (Pinto)
>            Reporter: Ron Sigal
>         Assigned To: Ron Sigal
>             Fix For: 2.2.2.GA_CP03, 2.4.0.Beta1 (Pinto)
>
>
> org.jboss.remoting.transport.socket.MicroSocketClientInvoker.getConnection() currently calls Thread.sleep(1000) after each unsuccessful attempt to retrieve a connection from the connection pool.  This introduced an unnecessary delay in general, and, in particular, makes the use of a connection pool of size one impractical.

-- 
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