-
1. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gsanderson Jan 7, 2009 9:18 PM (in response to gsanderson)the title which was truncated was supposed to "... passed from messaging to remoting layers".
I locally patched JMSRemotingConnection to copy the validatorPingTimeout and validatorPingPeriod from the URI into the config map, and everything worked as expected (i.e. the remoting-bisocket-service.xml values from the server were used on the client for the connection validation). -
2. Re: validatorPingPeriod/Timeout etc. not properly passed fro
timfox Jan 8, 2009 6:39 AM (in response to gsanderson)Howard- could you verify this and whether or not it is fixed in the latest remoting release?
-
3. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gaohoward Jan 8, 2009 6:44 AM (in response to gsanderson)sure I'll verify.
-
4. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gaohoward Jan 9, 2009 7:31 AM (in response to gsanderson)This should be a JBM issue. The client configuration parameters is not passed to the ConnectionValidator in jboss remoting, which performs the client ping operation.
In JMSRemotingConnection.addConnectionListener() and addPlainConnectionListener(), it uses client.addConnectionListener(listener), which doesn't pass in the config. Rather it should use the client.addConnectionListener(listener, metadata) so the configuration can get populated into remoting layer.
I'll create a JIRA issue for that.
Thanks -
5. Re: validatorPingPeriod/Timeout etc. not properly passed fro
ron_sigal Jan 9, 2009 5:10 PM (in response to gsanderson)"gsanderson" wrote:
but it is hard to be 100% sure.
I know what you mean. :)"gsanderson" wrote:
It appears the intention of the design is that the caller should populate the Map with all relevant parameters rather than having the remoting layer automatically pull them from the URI
Actually, org.jboss.remoting.Client passes a reference to itself to the ConnectionValidator, which then has access to the Client's configuration map, which includes the InvokerLocator (URI) parameters. ConnectionValidator will take parameters from the Client's configuration map and from the metadata map passed to Client.addConnectionListener(), with the later taking precedence. The problem prior to JBREM-1069 "Make ConnectorValidator configure ping period correctly" (https://jira.jboss.org/jira/browse/JBREM-1069) was that the single argument version of Client.addConnectionListener() put the default value for "validatorPingPeriod" in the metadata map, which meant that the default value would override the value in the InvokerLocator. With the fix to JBREM-1069, the value of "validatorPingPeriod" in the InvokerLocator should take effect. The value of "validatorPingTimeout" should already be taken from the InvokerLocator."gaohoward" wrote:
This should be a JBM issue. The client configuration parameters is not passed to the ConnectionValidator in jboss remoting, which performs the client ping operation.
As much as I'm happy to put the blame on JBM, I think it's a Remoting issue. This might be a first. Ask Tim. :) -
6. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gsanderson Jan 9, 2009 8:08 PM (in response to gsanderson)
Actually, org.jboss.remoting.Client passes a reference to itself to the ConnectionValidator, which then has access to the Client's configuration map, which includes the InvokerLocator (URI) parameters
The problem (or at least the one I fixed locally) is that the client configuration map is constructed from scratch by JMSRemotingConnection and doesn't contain the validatorPingTimeout or validatorPingPeriod from the URI at all.
https://jira.jboss.org/jira/browse/JBREM-1069 is required to avoid the remoting layer overwriting it once it actually makes it in ;-) -
7. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gsanderson Jan 9, 2009 8:13 PM (in response to gsanderson)actually now I'm confused... I still don't know who is responsible for the client configuration map contents.
I'll let you guys who know what you're talking about figure it out ;-) -
8. Re: validatorPingPeriod/Timeout etc. not properly passed fro
ron_sigal Jan 9, 2009 9:48 PM (in response to gsanderson)"gsanderson" wrote:
actually now I'm confused...
With good reason. I lied to you."gsanderson" wrote:
org.jboss.remoting.Client passes a reference to itself to the ConnectionValidator, which then has access to the Client's configuration map, which includes the InvokerLocator (URI) parameters.
That, the part about the InvokerLocator parameters, just isn't true. I was lost in my own blind spot. I've created JBREM-1082 "Allow ConnectionValidator to access InvokerLocator parameters" (https://jira.jboss.org/jira/browse/JBREM-1082) to resolve the problem.
In the meantime, a workaround available to JBM, if they haven't finished their current CP release, would be to go ahead with JBMESSAGING-1482 "Client remoting configuration not properly passed to jboss remoting" (https://jira.jboss.org/jira/browse/JBMESSAGING-1482).
Apologies all around.
-Ron -
9. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gaohoward Jan 11, 2009 10:30 PM (in response to gsanderson)If JBREM-1082 is done, the problem should be solved. Otherwise we need JBM-1482 done.
As JBM's next cp is using JBR sp11, so we need 1482 be fixed. -
10. Re: validatorPingPeriod/Timeout etc. not properly passed fro
ron_sigal Jan 12, 2009 3:31 PM (in response to gsanderson)"gaohoward" wrote:
As JBM's next cp is using JBR sp11, so we need 1482 be fixed.
Howard, will JBMESSAGING-1482 make it into the next EAP CP release? -
11. Re: validatorPingPeriod/Timeout etc. not properly passed fro
gaohoward Jan 12, 2009 11:29 PM (in response to gsanderson)Yes Ron. For EAP cp04.
-
12. Re: validatorPingPeriod/Timeout etc. not properly passed fro
rkonjeti Feb 5, 2009 12:01 PM (in response to gsanderson)I see many issues/bugs related to default pingPeriod and pingTimeout very low. There are bugs in JBossMessaging, etc that uses BiSocket.
If so, why dont we just change the defaults for ConnectionValidator to be 10 seconds and 5 seconds to begin with...Ofcourse, allow anybody to override it, if they need to. But, the defaults should be meaningful to begin with. -
13. Re: validatorPingPeriod/Timeout etc. not properly passed fro
timfox Feb 5, 2009 1:07 PM (in response to gsanderson)It's already been done