On 23 févr. 2011, at 18:13, Sanne Grinovero wrote:
 I've been trying to port current work in progress as dependency
of
 Infinispan Query,
 I'm wishing for these changes:
 
 1) TimeoutManager should throw specific exceptions directly, depending
 on the framework being used
 (see the pull request I sent you [1], I think it solves the issue, we
 might want to polish it a bit to have the JPA interface define it's
 own specific factory. 
I'll have a look
 
 2) The org.hibernate.search.engine.Loader interface:
 I'd just remove the init() method from the interface.
 Infinispan Query needs quite different types and parameters (and
 definitely not a Session), and I see no reason to have the init method
 expressed by the interface contract.
 Each framework should know how to create and initialize his specific loaders. 
Loader is not a public API. If we ever expose it, that's going to be fore Hibernate
Core's object loading.
In other words, I have the feeling that Loader cannot be reused between various object
sources. Does ISPN's Query module use them? Did you manager to reuse much code in
practice?