-
1. Re: New wireformat
clebert.suconic Feb 1, 2007 3:02 AM (in response to timfox)I see that you're calling remoting directly on DelegateSupport now for the new wire format. (so.. I think that means we don't have an interceptor for doing remote calls any more, right?)
How FailoverValveInterceptor is being involved on the invocation now? Is it still capturing exceptions?
I had the impression by briefly looking on code that FailoverValveInterceptor is inactive on this new codebase, as doInvoke methods are only throwing JMSException, and we will have to add the FailoverValve back somehow. -
2. Re: New wireformat
timfox Feb 1, 2007 4:54 AM (in response to timfox)"clebert.suconic@jboss.com" wrote:
I see that you're calling remoting directly on DelegateSupport now for the new wire format. (so.. I think that means we don't have an interceptor for doing remote calls any more, right?)
How FailoverValveInterceptor is being involved on the invocation now? Is it still capturing exceptions?
I had the impression by briefly looking on code that FailoverValveInterceptor is inactive on this new codebase, as doInvoke methods are only throwing JMSException, and we will have to add the FailoverValve back somehow.
This is a good point. I will investigate today. -
3. Re: New wireformat
timfox Feb 1, 2007 10:05 AM (in response to timfox)I am now catching network exceptions in DelegateSupport:
else if ((t instanceof CannotConnectException) || (t instanceof IOException)) { return new MessagingNetworkFailureException((Exception)t); }
And wrapping them in a JMSException subtype so they can reach the FailoverInterceptor.
This seems to work. Well most if the clustering tests run through now, although I am getting spurious failures where the test just times out and no failover seems to have occurred. Need to investigate further. -
4. Re: New wireformat
clebert.suconic Feb 1, 2007 10:48 AM (in response to timfox)Just as a brain storm...
I had the impression we could refactor failover a bit, to be called directly from these new methods.
Maybe adding new methods to FailoverCommandCenter.
As we are interested in capturing remoting failures only, we could have the valve and failure detection right on top of the invocation.. maybe having a hook to add such behavior on DelegateSupport. -
5. Re: New wireformat
clebert.suconic Feb 1, 2007 12:44 PM (in response to timfox)I've run the clustering testsuite...
and the only failure I get now is FailoverTest::testFailoverMessageOnServer
It looks like we need a server valve, or some sort of lock on the server side, for while the serverSide failover is being executed. -
6. Re: New wireformat
timfox Feb 1, 2007 1:00 PM (in response to timfox)"clebert.suconic@jboss.com" wrote:
I've run the clustering testsuite...
and the only failure I get now is FailoverTest::testFailoverMessageOnServer
It looks like we need a server valve, or some sort of lock on the server side, for while the serverSide failover is being executed.
We already have a server side valve. Were you not aware of this? -
7. Re: New wireformat
clebert.suconic Feb 1, 2007 1:44 PM (in response to timfox)I guess I was coding under the influence (CUI)
American's have an acronym for Driving Under the Influence - DUI... I guess that has happened to me on coding :-)