Re: [hibernate-dev] Regression after upgrade to 4.1.7.Final

2012-09-14 Thread Shawn Clowater
Spent some time on this. It's not my change, it's the change that allows for collections to be loaded outside of a session. In AbstractPersistentCollection boolean isExtraLazy = withTemporarySessionIfNeeded( new LazyI

Re: [hibernate-dev] Regression after upgrade to 4.1.7.Final

2012-09-14 Thread Shawn Clowater
I'm going to be rolling up to 4.1.7 sometime today since we just started our new release and I'm trying to stay as current as I can. I'll see what I can see however, unless the search code is using nested sessions then the patch wouldn't affect it. Since you're already there, can you tell me i

Re: [hibernate-dev] Regression after upgrade to 4.1.7.Final

2012-09-14 Thread Sanne Grinovero
I could reproduce the same problem by just running the Hibernate Search testsuite, from master just changing the hibernate-core dependency from 4.1.6 to 4.1.7 The failing test is org.hibernate.search.test.embedded.EmbeddedTest in the "hibernate-search-orm" Maven module, and reproduces exactly the

Re: [hibernate-dev] [OGM] Rethinking of naming around the map datastore

2012-09-14 Thread Guillaume SCHEIBEL
Oh really big modification with a lot of impacts :) But I think you can do that ;) Seriously, go ahead Guillaume 2012/9/14 Emmanuel Bernard > The Map datastore provider is named MapBasedDatastoreProvider and his > short cut is mapbased. > > To me it looks weird to use the suffix based. We don'

[hibernate-dev] [OGM] Rethinking of naming around the map datastore

2012-09-14 Thread Emmanuel Bernard
The Map datastore provider is named MapBasedDatastoreProvider and his short cut is mapbased. To me it looks weird to use the suffix based. We don't use InfinispanBasedDatastoreProvider for example. I'm inclined to: - rename MapBasedDatastoreProvider to MapDatastoreProvider - rename mapbased to m