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/co...
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 [sanne(a)hibernate.org]
>>> Received: Saturday, 10 Nov 2012, 2:56pm
>>> To: Scott Marlow [smarlow(a)redhat.com]
>>> CC: Emmanuel Bernard [emmanuel(a)hibernate.org]; Hibernate Dev
[hibernate-dev(a)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 <smarlow(a)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(a)lists.jboss.org
>>>>
https://lists.jboss.org/mailman/listinfo/hibernate-dev
>>
_______________________________________________
hibernate-dev mailing list
hibernate-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev