-
1. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 May 7, 2008 9:51 AM (in response to alrubinger)I changed the resolution logic in the container to always delegate to the DeploymentScope, so there always is one now. Previously this only existed for ears. As to whether the base name should be empty for non-ears is a detail that can be changed.
-
2. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 May 7, 2008 12:34 PM (in response to alrubinger)I changed the JBoss5DeploymentScope to use an empty "" base name if the deployment is not an ear, so that should help with legacy expectations for the jndi name.
-
3. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 May 7, 2008 12:38 PM (in response to alrubinger)Whoops, didn't read the logic carefully enough. Now its null if its not an ear.
-
4. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
alrubinger May 7, 2008 5:54 PM (in response to alrubinger)Scott's fixes have led to ~113 regressions resolved in EJB3 TestSuite:
http://jboss.hudson.alrubinger.com/job/EJB3_Plugin_AS_TRUNK/116/
S,
ALR -
5. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
wolfc May 8, 2008 4:21 AM (in response to alrubinger)Not reproducible here and on QA.
-
6. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
wolfc Jun 10, 2008 10:12 AM (in response to alrubinger)This is becoming ugly.
A deployment scope was meant to denote a JavaEEApplication (/ EAR) and thus it implements it as such. If the usage has changed we should at least disconnect DeploymentScope from JavaEEApplication.
Furthermore the determination of whether there is an EAR or not appears to be broken. As the jboss-seam-dvd application gets deployed with a deployment scope with a null base name. -
7. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
brian.stansberry Jun 10, 2008 10:35 AM (in response to alrubinger)See also related discussion at http://www.jboss.com/index.html?module=bb&op=viewtopic&t=135477
-
8. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 Jun 10, 2008 3:23 PM (in response to alrubinger)"wolfc" wrote:
This is becoming ugly.
A deployment scope was meant to denote a JavaEEApplication (/ EAR) and thus it implements it as such. If the usage has changed we should at least disconnect DeploymentScope from JavaEEApplication.
Its a resolution context/api now. Is the JavaEEApplication still useful?"wolfc" wrote:
Furthermore the determination of whether there is an EAR or not appears to be broken. As the jboss-seam-dvd application gets deployed with a deployment scope with a null base name.
I'll have to see why that is. I see Ales has added back in a hack to check the deployment unit suffix for ".ear" to get the jboss-seam-dvd application to work. -
9. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
alesj Jun 10, 2008 3:38 PM (in response to alrubinger)"scott.stark@jboss.org" wrote:
I see Ales has added back in a hack to check the deployment unit suffix for ".ear" to get the jboss-seam-dvd application to work.
That was Carlo. -
10. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
alesj Jun 10, 2008 3:56 PM (in response to alrubinger)"alesj" wrote:
"scott.stark@jboss.org" wrote:
I see Ales has added back in a hack to check the deployment unit suffix for ".ear" to get the jboss-seam-dvd application to work.
That was Carlo.
But it was a good hack. :-)
We at least saw that we're on the right path with making Seam run in AS5. -
11. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 Jun 10, 2008 5:19 PM (in response to alrubinger)The main complication is that the Ejb3Deployer is processing an ear deployment unit for essentially no reason. In the past I guess this is what was creating the root ejb3 DeploymentUnit to be able to resolve nested ejbs, but this is handled by other deployers.
Its also still doing too much processing that prepares for annotation scanning that is also done elsewhere. At some point this needs to be overhauled. For now I have just corrected the check for a nested/scoped ejb jar and the seam booking testcase is running. -
12. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
wolfc Jun 10, 2008 5:32 PM (in response to alrubinger)That has to do with persistence units at ear level.
As for the deployment scope or some sort of JavaEEApplication, that should come from some EAR deployer, not the Ejb3Deployer. -
13. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
starksm64 Jun 10, 2008 5:55 PM (in response to alrubinger)"wolfc" wrote:
That has to do with persistence units at ear level.
Can't that be moved out?"wolfc" wrote:
As for the deployment scope or some sort of JavaEEApplication, that should come from some EAR deployer, not the Ejb3Deployer.
Easy to do. -
14. Re: DeploymentScopeBaseName from JAR prepending JNDI Name
wolfc Jun 11, 2008 2:55 AM (in response to alrubinger)I think it must be moved out to a separate JPA deployer, which does persistence deployment for ear, war and jar.
This should not be done in AS trunk, it should happen in a new project. In trunk there should be a switch to turn off persistence deployments, which should then be picked up by the new JPA deployer. That allows for a gradual transition.
The interesting integration point will be the persistence unit injection facilitator.