[jboss-jira] [JBoss JIRA] Commented: (JBREM-757) Implement quick Client.removeListener() for polled callbacks.

[Ron Sigal (JIRA)]

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

The test method additions did not properly test polled callbacks.

Now, testQuickRemoveListener has been split into testQuickRemovePushListener() and testQuickRemovePullListner, and testQuickRemoveListenerZeroTimeout has been split into testQuickRemovePushListenerZeroTimeout and testQuickRemovePullListenerZeroTimeout.  For transports that are not bidirectional, quick versions of Client.disconnect() and quick Client.removeListener()  will be tested for polled callbacks.

Also, unit tests have been added for the bisocket transport.

> Implement quick Client.removeListener() for polled callbacks.
> -------------------------------------------------------------
>
>                 Key: JBREM-757
>                 URL: http://jira.jboss.com/jira/browse/JBREM-757
>             Project: JBoss Remoting
>          Issue Type: Feature Request
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 2.4.0.Beta1 (Pinto), 2.2.0.SP4, 2.2.1.GA
>            Reporter: Ron Sigal
>         Assigned To: Ron Sigal
>             Fix For: 2.4.0.Beta1 (Pinto), 2.2.2.GA
>
>
> For JBREM-657, quick org.jboss.remoting.Client.disconnect() and quick Client.removeListener() facilities were implemented which impose a per invocation timeout on any network i/o that occurs during these methods.  The per invocation timeout is set by calling Client.setDisconnectTimeout().  When the disconnect timeout is set to 0, network i/o is avoided entirely.
> However, the quick version of Client.removeListener() was implemented only for push callbacks.  It should also be implemented for pull callbacks.

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