Which failure do we want to address? 1. The integrator approach is not helping when using OGM/ORM on AS7.
2. The earlier failure that I haven't yet seen when the place holder is removed and we bootstrap with HibernateOgmPersistence.createContainerEntityManagerFactory(). Something else? On 11/13/2012 09:20 AM, Emmanuel Bernard wrote: > If you remove the place holder it will fail as well. Earlier actually, > so no. > On Tue 2012-11-13 8:20, Scott Marlow wrote: >> Let me know if you want me to try changing the OGM code that sets >> "---PlaceHolderDSForOGM---" to only set the datasource if its not >> already specified. >> >> For resolving why the OGM integrator being on the ORM classpath, is >> not enough to avoid the ORM datasource lookup, I'm not sure. >> >> On 11/12/2012 01:43 PM, Scott Marlow wrote: >>> Hardy pointed me to >>> https://github.com/hferentschik/ogm-kitchensink/blob/master/.openshift/config/modules/org/hibernate/ogm/module.xml >>> which duplicates some other AS modules jars in the OGM module. That is >>> going to be specific to which version of AS is being targeted (IMO) and >>> not something I recommend that we depend on. >>> >>> On 11/12/2012 12:46 PM, Emmanuel Bernard wrote: >>>> Hum, yes make sure to check the pom. There has been a few changes to >>>> some superclasses we extend. I did some changes in master for that. >>> >>> I was able to build OGM master against ORM 4.1.6 without getting build >>> errors. >>> >>>> >>>> Emmanuel >>>> >>>> On Mon 2012-11-12 12:00, Scott Marlow wrote: >>>>> Does it matter which version of Hibernate ORM is used (4.1.6 is what >>>>> we are currently using on AS master branch)? >>>>> >>>>> Which version of OGM should I try to use? Previously, I built OGM >>>> >from git master but I noticed that there is a beta1 available also. >>>>> >>>>> On 11/10/2012 05:41 PM, Scott Marlow wrote: >>>>>> Hi Sanne, >>>>>> >>>>>> I'm not sure why the module names didn't show for ogm/orm. >>>>>> >>>>>> I'll try again with the demo app. >>>>>> >>>>>> Scott >>>>>> >>>>>> >>>>>> Sent from my tardis >>>>>> >>>>>> >>>>>> -----Original Message----- >>>>>> From: Sanne Grinovero [sa...@hibernate.org] >>>>>> Received: Saturday, 10 Nov 2012, 2:56pm >>>>>> To: Scott Marlow [smar...@redhat.com] >>>>>> CC: Emmanuel Bernard [emman...@hibernate.org]; Hibernate Dev >>>>>> [hibernate-dev@lists.jboss.org] >>>>>> Subject: Re: [hibernate-dev] Documentation on OGM + JBoss AS >>>>>> >>>>>> >>>>>> Hi Scott, >>>>>> we don't have that error when testing on AS 7.1.1 with the >>>>>> kitchensink-ogm demo [1] so I'm confident the error is caused by some >>>>>> difference in the module; indeed I remember it being tricky to put the >>>>>> right things together: I can't test it on 7.2-SNAPSHOT now. >>>>>> >>>>>> We definitely don't specify any datasource in the demo configuration, >>>>>> but a custom module is required. >>>>>> >>>>>> Side question on the stacktrace you posted: why is the version number >>>>>> missing in the stack for Hibernate packages? Do we need to add that in >>>>>> the manifest? >>>>>> >>>>>> Sanne >>>>>> >>>>>> 1 - latest version at https://github.com/hferentschik/ogm-kitchensink >>>>>> >>>>>> On 9 November 2012 20:08, Scott Marlow <smar...@redhat.com> wrote: >>>>>>> On 11/09/2012 11:37 AM, Emmanuel Bernard wrote: >>>>>>>> On Fri 2012-11-09 11:15, Scott Marlow wrote: >>>>>>>>> On 10/01/2012 11:12 AM, Emmanuel Bernard wrote: >>>>>>>>>> On Mon 2012-10-01 10:51, Scott Marlow wrote: >>>>>>>>>>> I got further yesterday (after removing the >>>>>>>>>>> ---PlaceHolderDSForOGM--- property from HibernateOgmPersistence). >>>>>>>>>>> Not sure what the long term change is (perhaps we should only add >>>>>>>>>>> the ---PlaceHolderDSForOGM--- for the non-container case). >>>>>>>>>> >>>>>>>>>> That may have changed but this place holder was mandatory to get OGM >>>>>>>>>> to start in some environment. >>>>>>>>>> At least make sure that these change does not break the SE mode. >>>>>>>>>> >>>>>>>>> >>>>>>>>> People are still hitting the "Caused by: >>>>>>>>> org.hibernate.service.jndi.JndiException: Unable to lookup JNDI name >>>>>>>>> [---PlaceHolderDSForOGM---]" error. >>>>>>>>> >>>>>>>>> Since people are still hitting the ---PlaceHolderDSForOGM--- induced >>>>>>>>> failure (on AS7), we should create a jira for addressing it. The >>>>>>>>> fix could be to only add ---PlaceHolderDSForOGM--- when booting in >>>>>>>>> standalone mode but not container. >>>>>>>> >>>>>>>> The placeholder is there because otherwise, some part of Hibernate >>>>>>>> EntityManager's configuration bootstrap complain. >>>>>>>> >>>>>>>> But this value should never be used because Hibernate OGM has an >>>>>>>> integrator that overrides the ConnectionProviderInitiator with a Noop >>>>>>>> version. >>>>>>>> The fact that Sergio's stacktrace shows >>>>>>>> >>>>>>>> Caused by: org.hibernate.service.jndi.JndiException: Unable to lookup >>>>>>>> JNDI name [---PlaceHolderDSForOGM---] >>>>>>>> at >>>>>>>> org.hibernate.service.jndi.internal.JndiServiceImpl.locate(JndiServiceImpl.java:68) >>>>>>>> at >>>>>>>> org.hibernate.service.jdbc.connections.internal.DatasourceConnectionProviderImpl.configure(DatasourceConnectionProviderImpl.java:116) >>>>>>>> >>>>>>>> is proof to me that the OgmIntegrator is not taken into account which >>>>>>>> means that the file >>>>>>>> META-INF/services/org.hibernate.integrator.spi.Integrator >>>>>>>> is not seen by Hibernate ORM when it bootstraps the application which >>>>>>>> means that Hibernate OGM's classpath is not visible where it should be. >>>>>>> >>>>>>> I tried adding the Hibernate OGM module to the ORM module classpath and >>>>>>> still get the same error (http://pastie.org/5352457). >>>>>>> >>>>>>>> >>>>>>>> That's why I strongly suspects that this is either a JBoss modules bug >>>>>>>> or some misconfiguration of the ogm modules. >>>>>>>> >>>>>>>> We could try and give a better error report, open an issue for that but >>>>>>>> at the end of the day, something is broken in the ORM-AS integration >>>>>>>> wrt >>>>>>>> integrator and service discovery. >>>>>>>> >>>>>>>> Emmanuel >>>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> hibernate-dev mailing list >>>>>>> hibernate-dev@lists.jboss.org >>>>>>> https://lists.jboss.org/mailman/listinfo/hibernate-dev >>>>> >>> >>> _______________________________________________ >>> hibernate-dev mailing list >>> hibernate-dev@lists.jboss.org >>> https://lists.jboss.org/mailman/listinfo/hibernate-dev >>> >> _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev