[jboss-remoting-issues] [JBoss JIRA] Commented: (JBREM-1069) Make ConnectorValidator configure ping period correctly
[
https://jira.jboss.org/jira/browse/JBREM-1069?page=com.atlassian.jira.plu...
]
Ron Sigal commented on JBREM-1069:
----------------------------------
Changed org.jboss.remoting.Client.addConnectionListener(ConnectionListener listener) so
that it passes in ConnectionValidator.DEFAULT_PING_PERIOD only if the value of
ConnectionValidator.VALIDATOR_PING_PERIOD in the Client's configuration map has not
been set.
Changes applied to branches 2.2 and 2.x.
Unit test: Added testSetPingPeriodByClientConfigUsingSingleArgMethod() to
org.jboss.test.remoting.connection.ConnectionValidatorConfigTestClient.
Make ConnectorValidator configure ping period correctly
-------------------------------------------------------
Key: JBREM-1069
URL: https://jira.jboss.org/jira/browse/JBREM-1069
Project: JBoss Remoting
Issue Type: Bug
Security Level: Public(Everyone can see)
Affects Versions: 2.2.2.SP10, 2.5.0.SP2 (Flounder)
Reporter: Ron Sigal
Assignee: Ron Sigal
Fix For: 2.2.2.SP11, 2.5.0.SP3 (Flounder)
When the overloaded org.jboss.remoting.Client method
public void addConnectionListener(ConnectionListener listener)
is called, it puts the default value
org.jboss.remoting.ConnectionValidator.DEFAULT_PING_PERIOD (actual value 2000) in the
metadata map passed to ConnectionValidator. Since the metadata map overrides the
Client's configuration map, the value in the Client's configuration map, which can
come from the InvokerLocator or Connector configuration, is ignored.
In particular, the JBossMessaging class
org.jboss.jms.client.remoting.JMSRemotingConnection calls that version of
Client.addListener().
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira