Re: less-osgi-friendly code drop

2011-07-15 Thread David Blevins

On Jul 13, 2011, at 10:10 PM, David Jencks wrote:

 
 On Jul 13, 2011, at 9:46 PM, David Blevins wrote:
 
 
 On Jul 11, 2011, at 12:54 AM, David Jencks wrote:
 
 testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
 testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
 testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
 testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
 testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)
 
 Have all but 3 of these passing in the embedded container in my local copy.  
 The 3 that fail are broken ones.  Simply need to add validation for those. 
  My impl code is a bit hacked -- currently only works for @Stateful and not 
 exactly implemented in the right spot -- but otherwise looks like a good 
 approach.  Will clean it up and check it in tomorrow.
 
 
 
 Excellent!  On the geronimo side I've changed things around so that we have 
 one openejb appInfo tree for the entire application and use it to set up the 
 owb context.  This _ought_ to fix the problem in g. that I was seeing that 
 the owb context didn't include any of the specialized classes (since it was 
 looking at only one of the modules in the ear).  However I'm still cleaning 
 up loose ends so apps will deploy ok :-)

Checked in my specialization code to OWB and OpenEJB.  It fixes things in the 
embedded container (sans the three 'broken' tests). Still seeing them fail in G 
though.

Hacking up the validation logic for the 'broken' tests now.


-David



Re: less-osgi-friendly code drop

2011-07-14 Thread Shawn Jiang
Hi David Jencks,

Many cases in interop and webservices failed with following similar
exception at deployment phase after your owb/openejb refactor. Seems
deploymentinfo does not get initialized correclty in

org.apache.geronimo.openejb.EjbDeployment.initialize(BeanContext) {
...
}

at all.could you please take a look ?

Caused by: org.apache.geronimo.gbean.InvalidConfigurationException:
Configuration  xxx failed to start due
following reasons:
  The service
EJBModule=.jar,J2EEApplication=default//1-default/car,SingletonBean=,j
WSLink,name=  did not start because null
java.lang.NullPointerException
at
org.apache.geronimo.openejb.EjbDeployment.getBeanClass(EjbDeployment.java:239)
at
org.apache.geronimo.axis2.ejb.EJBWebServiceGBean.init(EJBWebServiceGBean.java:76)
at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native
Method)
at
sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
at
sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
at
org.apache.xbean.recipe.ReflectionUtil$ConstructorFactory.create(ReflectionUtil.java:958)
at
org.apache.xbean.recipe.ObjectRecipe.internalCreate(ObjectRecipe.java:276)
at
org.apache.xbean.recipe.AbstractRecipe.create(AbstractRecipe.java:96)
at
org.apache.xbean.recipe.AbstractRecipe.create(AbstractRecipe.java:61)
at
org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java:940)
at
org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:271)
at
org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:105)
at
org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:127)
at
org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:567)
at
org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:386)
at
org.apache.geronimo.kernel.config.ConfigurationUtil.startConfigurationGBeans(ConfigurationUtil.java:462)
at
org.apache.geronimo.kernel.config.KernelConfigurationManager.start(KernelConfigurationManager.java:226)
at
org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:702)
at
org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:681)
at
org.apache.geronimo.deployment.plugin.local.StartCommand.run(StartCommand.java:67)
at java.lang.Thread.run(Thread.java:619)



On Thu, Jul 14, 2011 at 1:10 PM, David Jencks david_jen...@yahoo.comwrote:


 On Jul 13, 2011, at 9:46 PM, David Blevins wrote:

 
  On Jul 11, 2011, at 12:54 AM, David Jencks wrote:
 
 
 testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
 
 testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 
 testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 
 testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
 
 testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
 
 testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
 
 testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)
 
  Have all but 3 of these passing in the embedded container in my local
 copy.  The 3 that fail are broken ones.  Simply need to add validation for
 those.  My impl code is a bit hacked -- currently only works for @Stateful
 and not exactly implemented in the right spot -- but otherwise looks like a
 good approach.  Will clean it up and check it in tomorrow.
 
 

 Excellent!  On the geronimo side I've changed things around so that we have
 one openejb appInfo tree for the entire application and use it to set up the
 owb context.  This _ought_ to fix the problem in g. that I was seeing that
 the owb context didn't include any of the specialized classes (since it was
 looking at only one of the modules in the ear).  However I'm still cleaning
 up loose ends so apps will deploy ok :-)

 thanks
 david jencks


  -David
 




-- 
Shawn


Re: less-osgi-friendly code drop

2011-07-14 Thread David Jencks
I just committed something that along with fixing some owb integration problems 
seems to fix this specific error.  I think that in the code below some gbean 
dependencies got mixed up and the web service gbean was starting before the ejb 
module actually started.  I moved the start code back into EjbModuleImpl and 
don't see this NPE.

thanks
david jencks

On Jul 14, 2011, at 4:38 AM, Shawn Jiang wrote:

 Hi David Jencks,
 
 Many cases in interop and webservices failed with following similar exception 
 at deployment phase after your owb/openejb refactor. Seems deploymentinfo 
 does not get initialized correclty in 
 
 org.apache.geronimo.openejb.EjbDeployment.initialize(BeanContext) {
 ...
 }
 
 at all.could you please take a look ?
 
 Caused by: org.apache.geronimo.gbean.InvalidConfigurationException: 
 Configuration  xxx failed to start due
 following reasons:
   The service 
 EJBModule=.jar,J2EEApplication=default//1-default/car,SingletonBean=,j
 WSLink,name=  did not start because null
 java.lang.NullPointerException
 at 
 org.apache.geronimo.openejb.EjbDeployment.getBeanClass(EjbDeployment.java:239)
 at 
 org.apache.geronimo.axis2.ejb.EJBWebServiceGBean.init(EJBWebServiceGBean.java:76)
 at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native 
 Method)
 at 
 sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:39)
 at 
 sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:27)
 at java.lang.reflect.Constructor.newInstance(Constructor.java:513)
 at 
 org.apache.xbean.recipe.ReflectionUtil$ConstructorFactory.create(ReflectionUtil.java:958)
 at 
 org.apache.xbean.recipe.ObjectRecipe.internalCreate(ObjectRecipe.java:276)
 at 
 org.apache.xbean.recipe.AbstractRecipe.create(AbstractRecipe.java:96)
 at 
 org.apache.xbean.recipe.AbstractRecipe.create(AbstractRecipe.java:61)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.createInstance(GBeanInstance.java:940)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.attemptFullStart(GBeanInstanceState.java:271)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.start(GBeanInstanceState.java:105)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstanceState.startRecursive(GBeanInstanceState.java:127)
 at 
 org.apache.geronimo.gbean.runtime.GBeanInstance.startRecursive(GBeanInstance.java:567)
 at 
 org.apache.geronimo.kernel.basic.BasicKernel.startRecursiveGBean(BasicKernel.java:386)
 at 
 org.apache.geronimo.kernel.config.ConfigurationUtil.startConfigurationGBeans(ConfigurationUtil.java:462)
 at 
 org.apache.geronimo.kernel.config.KernelConfigurationManager.start(KernelConfigurationManager.java:226)
 at 
 org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:702)
 at 
 org.apache.geronimo.kernel.config.SimpleConfigurationManager.startConfiguration(SimpleConfigurationManager.java:681)
 at 
 org.apache.geronimo.deployment.plugin.local.StartCommand.run(StartCommand.java:67)
 at java.lang.Thread.run(Thread.java:619)
 
 
 
 On Thu, Jul 14, 2011 at 1:10 PM, David Jencks david_jen...@yahoo.com wrote:
 
 On Jul 13, 2011, at 9:46 PM, David Blevins wrote:
 
 
  On Jul 11, 2011, at 12:54 AM, David Jencks wrote:
 
  testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
  testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
  testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
  testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
  testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)
 
  Have all but 3 of these passing in the embedded container in my local copy. 
   The 3 that fail are broken ones.  Simply need to add validation for 
  those.  My impl code is a bit hacked -- currently only works for @Stateful 
  and not exactly implemented in the right spot -- but otherwise looks like a 
  good approach.  Will clean it up and check it in tomorrow.
 
 
 
 

Re: less-osgi-friendly code drop

2011-07-13 Thread David Blevins

On Jul 11, 2011, at 12:54 AM, David Jencks wrote:

  
 testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
  
 testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  
 testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  
 testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
  
 testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
  
 testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
  
 testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)

Have all but 3 of these passing in the embedded container in my local copy.  
The 3 that fail are broken ones.  Simply need to add validation for those.  
My impl code is a bit hacked -- currently only works for @Stateful and not 
exactly implemented in the right spot -- but otherwise looks like a good 
approach.  Will clean it up and check it in tomorrow.


-David



Re: less-osgi-friendly code drop

2011-07-13 Thread David Jencks

On Jul 13, 2011, at 9:46 PM, David Blevins wrote:

 
 On Jul 11, 2011, at 12:54 AM, David Jencks wrote:
 
 testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
 testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
 testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
 testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
 testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
 testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)
 
 Have all but 3 of these passing in the embedded container in my local copy.  
 The 3 that fail are broken ones.  Simply need to add validation for those.  
 My impl code is a bit hacked -- currently only works for @Stateful and not 
 exactly implemented in the right spot -- but otherwise looks like a good 
 approach.  Will clean it up and check it in tomorrow.
 
 

Excellent!  On the geronimo side I've changed things around so that we have one 
openejb appInfo tree for the entire application and use it to set up the owb 
context.  This _ought_ to fix the problem in g. that I was seeing that the owb 
context didn't include any of the specialized classes (since it was looking at 
only one of the modules in the ear).  However I'm still cleaning up loose ends 
so apps will deploy ok :-)

thanks
david jencks


 -David
 



less-osgi-friendly code drop

2011-07-11 Thread David Jencks
I've back-ported the owb/jcdi related changes on the osgi-friendly branch to 
trunk and pushed a code drop to 

http://people.apache.org/~djencks/less-osgi-geronimo-3.0-SNAPSHOT-source-release.zip

So far all I know is that it compiles for me and the tomcat full server starts. 
 With the osgi-friendly code I was down to these jcdi results:

Failed tests: 
  
testRequestScopeDestroyedAfterCallToEjbTimeoutMethod(org.jboss.jsr299.tck.tests.context.request.ejb.EJBRequestContextTest)
  
testSessionContextDestroyedWhenHttpSessionTimesOut(org.jboss.jsr299.tck.tests.context.session.SessionContextTest)
  
testSessionContextSharedBetweenServletRequestsInSameHttpSession(org.jboss.jsr299.tck.tests.context.session.SessionContextTest)
  
testSessionContextDestroyedWhenHttpSessionInvalidated(org.jboss.jsr299.tck.tests.context.session.SessionContextTest)
  
testStereotypeWithNonEmptyNamed(org.jboss.jsr299.tck.tests.definition.stereotype.broken.nonEmptyNamed.NonEmptyNamedTest)
  
testStereotypeWithTooManyScopeTypes(org.jboss.jsr299.tck.tests.definition.stereotype.broken.tooManyScopes.TooManyScopeTypesTest)
  
test(org.jboss.jsr299.tck.tests.deployment.packaging.bundledLibrary.LibraryInEarTest)
  
testPrincipalBean(org.jboss.jsr299.tck.tests.implementation.builtin.BuiltInBeansTest)
  
testSerializeSFSB(org.jboss.jsr299.tck.tests.implementation.enterprise.lifecycle.EnterpriseBeanLifecycleTest)
  
testApplicationCannotCallRemoveMethodOnNonDependentScopedSessionEnterpriseBean(org.jboss.jsr299.tck.tests.implementation.enterprise.remove.EnterpriseBeanRemoveMethodTest)
  
testPassivationOfEjbs(org.jboss.jsr299.tck.tests.implementation.simple.resource.ejb.EjbInjectionTest)
  
testSpecializedBeanNotInstantiated(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationIntegrationTest)
  
testSpecializingBeanHasBindingsOfSpecializedAndSpecializingBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  
testSpecializingBeanHasNameOfSpecializedBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.EnterpriseBeanSpecializationTest)
  
testSpecializingClassDirectlyExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsNothing.DirectlyExtendsNothingTest)
  
testSpecializingClassDirectlyExtendsSimpleBean(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.directlyExtendsSimpleBean.DirectlyExtendsSimpleBeanTest)
  
testSpecializingClassImplementsInterfaceAndExtendsNothing(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.implementInterfaceAndExtendsNothing.ImplementsInterfaceAndExtendsNothingTest)
  
testSpecializingAndSpecializedBeanHasName(org.jboss.jsr299.tck.tests.inheritance.specialization.enterprise.broken.sameName.SameNameTest)
  
testNonExistantClassInBeansXmlNotOk(org.jboss.jsr299.tck.tests.interceptors.definition.broken.nonExistantClassInBeansXml.NonExistantClassInBeansXmlTest)
  
testNonContextualSessionBeanReferenceIsIntercepted(org.jboss.jsr299.tck.tests.interceptors.definition.enterprise.nonContextualReference.SessionBeanInterceptorOnNonContextualEjbReferenceTest)
  
testELResolverRegisteredWithServletContainer(org.jboss.jsr299.tck.tests.lookup.el.integration.IntegrationWithUnifiedELTest)

Tests run: 845, Failures: 21, Errors: 0, Skipped: 0

(note that one of the built-in bean tests that was working for a while isn't 
right now).

If I can convince myself this is unlikely to have broken much I'll probably 
commit to svn tomorrow if it won't get in dblevin's way.

thanks
david jencks