Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-06 Thread Guillaume Smet
+1 On Fri, Dec 6, 2013 at 1:15 PM, Sanne Grinovero wrote: > Hi all, > sorry was traveling yesterday, and while I saw & merged Guillaume's > pull request, I missed this conversation. > > We can to an Hibernate Search 4.4 minor release, would that resolve > all uncomforts? > > The release blog woul

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-06 Thread Gunnar Morling
+1 Sounds reasonable. 2013/12/6 Sanne Grinovero > Hi all, > sorry was traveling yesterday, and while I saw & merged Guillaume's > pull request, I missed this conversation. > > We can to an Hibernate Search 4.4 minor release, would that resolve > all uncomforts? > > The release blog would then a

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-06 Thread Sanne Grinovero
Hi all, sorry was traveling yesterday, and while I saw & merged Guillaume's pull request, I missed this conversation. We can to an Hibernate Search 4.4 minor release, would that resolve all uncomforts? The release blog would then also be an appropriate medium to warn about the incompatibility. S

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-06 Thread Guillaume Smet
Is there any decision taken about this? I find the situation to have 2 major Hibernate components incompatible after an upgrade to a point release a bit uncomfortable. The cleanest way to fix it is probably to build a point release of Search but if it's too much work, maybe we should just communi

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Guillaume Smet
On Thu, Dec 5, 2013 at 5:31 PM, Guillaume Smet wrote: > I'm working on a pull request to fix the problem in search as > requested by Steve. Done: https://github.com/hibernate/hibernate-search/pull/521 -- Guillaume ___ hibernate-dev mailing list hibern

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Guillaume Smet
- > From: "Guillaume Smet" > To: "Scott Marlow" > Cc: "Hibernate" > Sent: Thursday, December 5, 2013 11:09:42 AM > Subject: Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and > thus HSearch > > Yes. This commit looks really nice

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Brett Meyer
er Red Hat, Hibernate ORM - Original Message - From: "Guillaume Smet" To: "Scott Marlow" Cc: "Hibernate" Sent: Thursday, December 5, 2013 11:09:42 AM Subject: Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch Yes. This com

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Steve Ebersole
Can't (read you should be!) you use org.hibernate.engine.spi.SessionImplementor#generateEntityKey ? Its signature hides the values from Session actually kept on the genernated EntityKey: public EntityKey generateEntityKey(Serializable id, EntityPersister persister); That signature has not ch

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Guillaume Smet
Yes. This commit looks really nice: that's what looks really interesting to me in 4.2.8. But providing a compatibility layer seems necessary. On Thu, Dec 5, 2013 at 5:06 PM, Scott Marlow wrote: > Looks like this commit changed that > https://github.com/hibernate/hibernate-orm/commit/bf26311474257

Re: [hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Scott Marlow
Looks like this commit changed that https://github.com/hibernate/hibernate-orm/commit/bf26311474257c2f0118615e003553095c2d87b0 On 12/05/2013 10:51 AM, Guillaume Smet wrote: > Hi all, > > ORM 4.2.8.Final breaks the API of EntityKey as it removes tenantId > from the constructor. > > Typically, in H

[hibernate-dev] ORM 4.2.8.Final breaks the EntityKey API and thus HSearch

2013-12-05 Thread Guillaume Smet
Hi all, ORM 4.2.8.Final breaks the API of EntityKey as it removes tenantId from the constructor. Typically, in HSearch, we have the following call: https://github.com/hibernate/hibernate-search/blob/master/orm/src/main/java/org/hibernate/search/query/hibernate/impl/PersistenceContextObjectsInitia