> -----Original Message----- > From: [EMAIL PROTECTED] > [mailto:[EMAIL PROTECTED]]On Behalf Of Sacha > Labourey > Sent: Friday, February 14, 2003 2:17 AM > To: [EMAIL PROTECTED] > Subject: RE: [JBoss-dev] TxInterceptor split is really really good > > > > > Why is this unfortunate? I thought it was a great idea that is long > > > overdue. I don't see the point in doing it 2 or three times, > > so I'd do it > > > after all the interceptors are based on a common interface/model. > > > > Yes, its long overdue. I'm doing it now....By "unfortunate" I > meant, this > > is a pain in the ass to do since it affects a lot of code and a > > lot of code > > must be retested. > > Which kind of model do you use? When there is an exception, do you simply > propagate it (and thus we keep the actual try ... finally construct) or do > you attach it to the InvocationResult and exceptions are never > brought back > as is? >
I propagate the exception. The change would have been a LOT harder if the Exception was placed within the InvocationResponse object so I didn't do it at first and will wait to see if a requirement comes up that needs this change. The original intent of this is to refactor the clustering code so that interceptors can be placed in the chain after the Clustering interceptor has chosen the invocation target. Bill > > > ------------------------------------------------------- > This SF.NET email is sponsored by: FREE SSL Guide from Thawte > are you planning your Web Server Security? Click here to get a FREE > Thawte SSL guide and find the answers to all your SSL security issues. > http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0026en > _______________________________________________ > Jboss-development mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/jboss-development ------------------------------------------------------- This SF.NET email is sponsored by: FREE SSL Guide from Thawte are you planning your Web Server Security? Click here to get a FREE Thawte SSL guide and find the answers to all your SSL security issues. http://ads.sourceforge.net/cgi-bin/redirect.pl?thaw0026en _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development