August 2008 - jboss-dev-forums - Jboss List Archives

[Design the new POJO MicroContainer] - Re: MDR doesn't work on annotated privates

by adrian＠jboss.org

"wolfc" wrote : | Yes you can, see EJB 3 12.4.1 the last bullet. | | It's just that if there is a private method with a @PostConstruct it must be invoked regardless of the sub class (which is the weird part). So I'm no longer understanding this issue. In my example would Sub::postConstruct() be invoked or not? The part of the EJB3 spec you refer to says the opposite of your "weird part" anonymous wrote : | If a lifecycle callback interceptor method is overridden by another method (regardless of | whether that method is itself a lifecycle callback interceptor method (of the same or different | type)), it will NOT be invoked. | My emphasis. So I don't see why you have to look at annotations on overridden methods. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170318#4170318 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170318

17 years, 7 months

1
0
0 / 0

[Design the new POJO MicroContainer] - Re: MDR doesn't work on annotated privates

by adrian＠jboss.org

"wolfc" wrote : "adrian(a)jboss.org" wrote : | | Then it would be configuratble which annotations you need to do | | the wangy search (no suprises or extra work for other annotations) | | and it would be transparent to the user of MetaData. | Doesn't this contradict the example you're about to give? | There the Sub.postConstruct override doesn't have the annotation, but if it's inheritable it would get one. | The point of the example is to show the problem. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170314#4170314 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170314

17 years, 7 months

1
0
0 / 0

[Design the new POJO MicroContainer] - Re: MDR doesn't work on annotated privates

by wolfc

"adrian(a)jboss.org" wrote : Why a new signature? The idea of the MetaData is that it should be able to | load relevant metadata and the caller doesn't have to think too hard. | | If we are going to support I'd prefer to have something like: | | | | <bean name="MetaDataRepository" ...> | | <property name="joinpointInheritableAnnotations"> | | <set class="java.lang.Class"> | | <value>javax.ejb.PostConstruct</value> | | </set> | | ... | | | | Then it would be configuratble which annotations you need to do | the wangy search (no suprises or extra work for other annotations) | and it would be transparent to the user of MetaData. Doesn't this contradict the example you're about to give? There the Sub.postConstruct override doesn't have the annotation, but if it's inheritable it would get one. That's not what I want. "adrian(a)jboss.org" wrote : Incidently, I think this semantic is broken, since it means you cannot | override the annotations of the super class. Not very OOPS. :-) | | | | public Sub extends Super | | { | | @Override | | // No annotations here | | public void postConstruct() {} | | | | // This is my postConstruct method | | @PostConstruct | | public void myPostConstruct() {} | | } | | Yes you can, see EJB 3 12.4.1 the last bullet. It's just that if there is a private method with a @PostConstruct it must be invoked regardless of the sub class (which is the weird part). View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170310#4170310 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170310

17 years, 7 months

1
0
0 / 0

[Design of POJO Server] - Re: Integrating aop-mc-int bean metadata with AS5

by alesj

"kabir.khan(a)jboss.com" wrote : | Modifying the code I copied from BeanMetaDataDeployer to | | | KernelControllerContext context = new AbstractKernelControllerContext(null, deployment, null); | | //Make sure that the metadata from the deployment gets put into the context | | ScopeInfo scopeInfo = context.getScopeInfo(); | | if (scopeInfo != null) | | { | | mergeScopes(scopeInfo.getScope(), unit.getScope()); | | //mergeScopes(scopeInfo.getMutableScope(), unit.getMutableScope()); | | } | | | | try | | { | | controller.install(context); | | } | | | This is old code. Adrian fixed this in JBMICROCONT-330. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170309#4170309 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170309

17 years, 7 months

1
0
0 / 0

[Design of POJO Server] - Re: Integrating aop-mc-int bean metadata with AS5

by kabir.khan＠jboss.com

"kabir.khan(a)jboss.com" wrote : | So that the scope is simply | | | [INSTANCE=ScopedAlias_17_Factory$ScopedInterceptor] | | | seems to get the aliases installed. | So that the alias is Should read: So that the mutable scope is simply | [INSTANCE=ScopedAlias_17_Factory$ScopedInterceptor] | seems to get the aliases installed. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170306#4170306 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170306

17 years, 7 months

1
0
0 / 0

[Design of POJO Server] - Re: Integrating aop-mc-int bean metadata with AS5

by kabir.khan＠jboss.com

Seems it was getting confused about the scopes. In the standalone version, the Bean MetaData annotations are added to the mutable scope by | [INSTANCE=SomeRandomName] | In the AS version the mutable scope was | [APPLICATION=vfszip:/Users/kabir/sourcecontrol/jboss-head/testsuite/output/lib/aop-scopeddependency-scoped.sar, INSTANCE=ScopedAlias_17_Factory$ScopedInterceptor] | When calling BasicKernelMetaDataRepository.getMetaData() with the context's scope | [JVM=THIS, CLASS=class org.jboss.aop.asintegration.jboss5.ScopedVFSClassLoaderDomain, INSTANCE=ScopedManager_4_vfszip:/Users/kabir/sourcecontrol/jboss-head/testsuite/output/lib/aop-scopeddependency-scoped.sar] | the returned MetaData does not contain the added annotations in AS. Modifying the code I copied from BeanMetaDataDeployer to | KernelControllerContext context = new AbstractKernelControllerContext(null, deployment, null); | //Make sure that the metadata from the deployment gets put into the context | ScopeInfo scopeInfo = context.getScopeInfo(); | if (scopeInfo != null) | { | mergeScopes(scopeInfo.getScope(), unit.getScope()); | //mergeScopes(scopeInfo.getMutableScope(), unit.getMutableScope()); | } | | try | { | controller.install(context); | } | So that the scope is simply | [INSTANCE=ScopedAlias_17_Factory$ScopedInterceptor] | seems to get the aliases installed. So that the alias is View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170305#4170305 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170305

17 years, 7 months

1
0
0 / 0

[Design the new POJO MicroContainer] - Re: MDR doesn't work on annotated privates

by adrian＠jboss.org

"wolfc" wrote : "adrian(a)jboss.org" wrote : | | I think we need a DeclaredMethodSignature. That should also solve the interfaces issue. Then I can put ejb3 meta data bridge on that one. | | Why a new signature? The idea of the MetaData is that it should be able to | load relevant metadata and the caller doesn't have to think too hard. | | If we are going to support I'd prefer to have something like: | | | | <bean name="MetaDataRepository" ...> | | <property name="joinpointInheritableAnnotations"> | | <set class="java.lang.Class"> | | <value>javax.ejb.PostConstruct</value> | | </set> | | ... | | | | Then it would be configuratble which annotations you need to do | the wangy search (no suprises or extra work for other annotations) | and it would be transparent to the user of MetaData. | | Incidently, I think this semantic is broken, since it means you cannot | override the annotations of the super class. Not very OOPS. :-) | | | | public Sub extends Super | | { | | @Override | | // No annotations here | | public void postConstruct() {} | | | | // This is my postConstruct method | | @PostConstruct | | public void myPostConstruct() {} | | } | | View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170304#4170304 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170304

17 years, 7 months

1
0
0 / 0

[Design the new POJO MicroContainer] - Re: MDR doesn't work on annotated privates

by adrian＠jboss.org

"wolfc" wrote : Well in actuality AnnotatedElementMetaDataLoader already works as I would expect it: http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas?view=rev&revision=76978. But because of the semantics of MethodSignature it's luck and not design that drives this (JBMDR-28 being the four-leaf clover). | It only works because I removed the validation that the Method was actually part of the class. I did that for performance reasons since it was showing up as a hotspot for AOP joinpoint analysis. I expect the callers to do the right thing, which you aren't. ;-) View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170303#4170303 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170303

17 years, 7 months

1
0
0 / 0

[Design of JBoss jBPM] - Re: With Ganymede the jPDL Designer has a strange appearance

by SidKennedy

ok thanks. but is there a possibility to fix this or to workaround? maybe this isn't the right place for asking that? View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170301#4170301 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170301

17 years, 7 months

1
0
0 / 0

[Design of JBoss Profiler] - Re: Does client and profiler have to point to localhost?

by jesper.pedersen

client -h hostname Please, read through README.txt for details. View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4170299#4170299 Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4170299

17 years, 7 months

1
0
0 / 0

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

jboss-dev-forums August 2008