Keyo, 

InvocationException extends Invocation
InvocationResponse extends Invocation

So you get the original context in both cases

marcf

> -----Original Message-----
> From: [EMAIL PROTECTED] 
> [mailto:[EMAIL PROTECTED] On 
> Behalf Of Sacha Labourey
> Sent: Saturday, September 20, 2003 11:34 AM
> To: [EMAIL PROTECTED]
> Subject: RE: [JBoss-dev] Re: [jboss-group] JRMPInvokerProxy
> 
> 
> > Unifying the interceptors would be a good idea.  The only 
> thing that 
> > needs to be "multiple" is AOP that deals with interfaces (Dynamic 
> > Proxy) or the AOP that deals with raw objects and the javassist
> > framework. But
> > definitely the SIGNATURE of the invocation should be unified so that
> > interceptors themselves come in ONE flavor 
> > 
> > Public Invocation invoke(Invocation)
> 
> Right now, the new signature that Bill implemented is something like:
> 
>       public InvocationResponse invoke (Invocation) throws Throwable
> 
> More specifically, when an exception is thrown, it is not 
> wrapped inside the InvocationResponse but thrown back 
> directly (not even wrapped). This is bad IMHO as this prevent 
> to attach payload to invocations that generate exceptions. If 
> we want to always be able to send back data to the client 
> side, we don't care if an Application Level exception (even 
> system in fact) has been thrown.
> 
> One way to solve that and if we want to keep the good old 
> (working) try...finally construct we have today in the 
> interceptors would be to change the signature like that:
> 
>       public InvocationResponse invoke (Invocation) throws 
> InvocationException
> 
> And have the InvocationException exception wrap the source 
> Exception/Error and allow attachement of arbitrary payload.
> 
> Cheers,
> 
> 
> 
>                       Sacha
> 
> 
> 
> 
> xxxxxxxxxxxxxxxxxxxxxxxxxxx
> Sacha Labourey
> General Manager
> JBoss Group (Europe) SàRL
> xxxxxxxxxxxxxxxxxxxxxxxxxxx
> 
> 
> 
> -------------------------------------------------------
> This sf.net email is sponsored by:ThinkGeek
> Welcome to geek heaven.
> http://thinkgeek.com/sf 
> _______________________________________________
> JBoss-Development mailing list [EMAIL PROTECTED]
> https://lists.sourceforge.net/lists/listinfo/jboss-development
> 



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
JBoss-Development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to