"rkadayam" wrote : "Bill Burke" wrote : 
  |   | 
  |   | You cannot remove an advice or interceptor from a binding.  You have to 
redeploy the whole binding.
  |   | 
  |   | If you have suggestions for expanding the API, feel free to play with the code 
and then submit it.   Or post your API change suggestions on a different topic.  You 
probably want an addInterceptor method on AspectManager.  There is an 
addInterceptorFactory.
  |   | 
  | 
  | I can understand the simplification in AdviceBinding to not maintain interceptor 
references explicitly but just array of interceptor factories to benefit performance. 
But even if I have to re-deploy the advice binding, I'd like to somehow pick out that 
one interceptor I want to remove. I noticed GenericInterceptorFactory has a 
"getClassName". Would it then make sense to add this method to InterfaceFactory? So 
that I could maybe parse the list of interceptor factories find the one that returns 
my interceptor class, remove it and recreate the AdviceBinding and add it to the 
AspectManager.
  | 
  | For now I could just cast the IFs to GenericInterceptorFactory.
  | 
  | I'm not too thrilled with the above approach but do you have any other suggestions 
without compromising performance?
  | 
  | Also would it make sense to allow an InterceptorFactory to be passed in 
AdviceBinding.addInterceptor() so that we can over-ride Generic if required? In fact 
over here I would much rather call it "attach" instead of "add". Or the alternative 
would be to have something like 
AspectManager.createInterceptor(name,class,deployment-model, interceptor factory)
  | 
  | AspectWerkz had a very similar method call like above except they did not have the 
concept of interceptor factory but allowed the "ClassLoader" itself to be passed as 
argument so you could practically load it from anywhere.
  | 
  | I envision interceptors to be attached and detached on regular basis. Consider 
someone trying to monitor some custom events using interceptors and they would be 
required to be easily swappable.
  | 
  | Thanks for all your help
  | rajiv

View the original post : 
http://www.jboss.org/index.html?module=bb&op=viewtopic&p=3828667#3828667

Reply to the post : 
http://www.jboss.org/index.html?module=bb&op=posting&mode=reply&p=3828667


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
JBoss-Development mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/jboss-development

Reply via email to