Re: [jboss-dev] jbossas -beans.xml opt

No, I think we're still at the level where you agree. :-) As this "not" can only be applied to AOPDependencyBuilder, not on the AOPConstructorJoinpoint. Or I think it's just the other way around. Kabir would know better. Somewhere AOP does kick in, as it knows how to add AOP's lifecycle callbacks to ControllerContext. Brian Stansberry wrote: > If not, that re-opens the discussion as to whether @DisableAOP > should be required, or should be default behavior with an > @EnableAOP. Since AFAICT the thing that made me thing enabling it > was required for the common case was @JMX. > Ales Justin wrote: >> I don't think we need AOPBD for this. Kabir? >> >> Hmmm, perhaps something like this should be on the @DisableAOP: >> @DisableAOP(allow={INSTANTIATE}) >> which would mean we don't use AOPDB - since the aspect we expect >> with this bean don't have any dependencies, >> but we do want AOP proxy instantiation, since we *do* expect the >> bean to be aspectized. >> >> Jason T. Greene wrote: >>> Ok, so then, why do we need AOPDependencyBuilder to use @JMX? Yes >>> I know the dep builder allows for users to add aspects after the >>> fact, but this is a near *useless* feature, especially for our >>> core components. >>> >>> Brian Stansberry wrote: >>>> The registration of the @JMX handling is in the bootstrap, in >>>> bootstrap/jmx.xml. >>>> >>>> Jason T. Greene wrote: >>>>> Ales Justin wrote: >>>>>> (2) @org.jboss.aop.microcontainer.annotations.DisableAOP >>>>>> >>>>>> This one instructs MC to ignore transparent AOP usage when >>>>>> handling your bean. >>>>>> It will not look for aspect dependencies or try to create an >>>>>> AOP proxy. >>>>>> It will simply fall back to plain POJO handling. >>>>>> >>>>>> If you use @JMX or anything similar, this should then *not* be >>>>>> used. >>>>>> But for anything else it should be good to use it. >>>>> >>>>> Can we fix the @JMX (and other known cases) by deploying them >>>>> sooner, as part of the bootstrap for example? >>>>> _______________________________________________ >>>>> jboss-development mailing list >>>>> jboss-development(a)lists.jboss.org >>>>> https://lists.jboss.org/mailman/listinfo/jboss-development >>>> >>>> >>> >>> _______________________________________________ >>> jboss-development mailing list >>> jboss-development(a)lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/jboss-development >>> >> _______________________________________________ >> jboss-development mailing list >> jboss-development(a)lists.jboss.org >> https://lists.jboss.org/mailman/listinfo/jboss-development _______________________________________________ jboss-development mailing list jboss-development(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/jboss-development