Re: [hibernate-dev] ConnectionObserver

2019-08-21 Thread Sanne Grinovero
Thanks all! Will incorporate these changes in my upcoming perfomance PR. On Wed, 21 Aug 2019 at 19:30, Steve Ebersole wrote: > > I'm fine with deprecating and removing. > > WRT changing `List` to just `ConnectionObserver` rather > we'd want to change it to `ConnectionObserverStatsBridge` to

Re: [hibernate-dev] ConnectionObserver

2019-08-21 Thread Steve Ebersole
I'm fine with deprecating and removing. WRT changing `List` to just `ConnectionObserver` rather we'd want to change it to `ConnectionObserverStatsBridge` to avoid the polymorphic call overhead. But otherwise I agree with everything On Tue, Aug 20, 2019 at 3:51 PM Gail Badner wrote: > In 4.2

[hibernate-dev] ConnectionObserver

2019-08-20 Thread Sanne Grinovero
While refactoring some related optimisations, I noticed there's currently no way to register a custom ConnectionObserver. There seems to be only one implementation, which is registered by default: - org.hibernate.internal.ConnectionObserverStatsBridge#ConnectionObserverStatsBridge Some