Hi,
just checked that it is already proxying all of:
org.hibernate.classic.Session.class,
org.hibernate.engine.SessionImplementor.class,
org.hibernate.jdbc.JDBCContext.Context.class,
org.hibernate.event.EventSource.class
so the patch is actually a one-liner to add the method name to the whitelist.
I've created HHH-4749 for this, if someone objects please comment there.
Also it would be usefull if we could apply this on 3.3 branch too as
it's affecting current Search users and I don't think existing code is
otherwise changing behaviour. WDYT?
I'll be offline the rest of this week for Seam courses, at
internet-less location :-/
Sanne
2010/1/4 Emmanuel Bernard <emman...@hibernate.org>:
> We are not proxying SessionImplementor but only Session.
> so we would need to
> - proxy SessionImplementor and Session
> - whitelist getListeners
>
> I don't see any issue to that. If nobody objects we can move forward.
>
> On 4 janv. 2010, at 01:16, Sanne Grinovero wrote:
>
>> Hello All,
>> when binding the Session to a thread (when using
>> org.hibernate.context.ThreadLocalSessionContext) it is currently
>> mandatory to start a transaction before any use of the session, this
>> includes currently Search.getFullTextSession, mandatory in order to
>> use Hibernate Search, as it uses getListener().
>> This behaviour was introduced since Search 3.1.1 to prevent other
>> trouble with code forgetting to start a transaction it will use
>> session.getListeners() as soon as the user calls getFullTextSession
>> (HSEARCH-178, which is a workaround for HHH-2763)
>>
>> The method "getListeners" is not in the whitelist of allowed methods
>> on the session; could this be added?
>> I'm referring to the private class TransactionProtectionWrapper in
>> org.hibernate.context.ThreadLocalSessionContext, around line 330.
>>
>> Could this also be applied to 3.3.x branch? it would help users of
>> Search using these core versions, while having no effect impact on
>> others uses.
>>
>> Regards,
>> Sanne
>> _______________________________________________
>> hibernate-dev mailing list
>> hibernate-dev@lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>
>
_______________________________________________
hibernate-dev mailing list
hibernate-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/hibernate-dev
