Re: [hibernate-dev] Extract some methods form AbtractEntityManagerImpl

I see a couple of options and a big fundamental question. All need feedback from the ORM gurus I am not sure AbstractEntityManager is a SPI in the first place. Naively I’d say it should be an internal class. Anyone remembers why it’s an SPI? 1. If AbstractEntityMangerImpl becomes internal, it’s not too problematic to open up the methods you are looking for and make them public. 2. Alternatively, you can move the method implementations in a helper class (static method maybe) and put this code in an internal package, you can then share these methods. I guess you can do this code change to both 4.3 and 5 and we will change the OGM code in the next released version. But that leads to an interesting question. We have API, SPI and internals. we all agree on the definition of internals, we have our differences when it comes to API vs SPI. But what we do not have is - a stable / tested contract between ORM and OGM / Search - that is not supported for regular ORM users Do you see this as doable and worthwhile? Maybe an annotation on the contract and some annotated tests would be enough? On 14 Feb 2014, at 17:32, Davide D'Alto <davide(a)hibernate.org&gt; wrote: