I'm fine with deprecating and removing. WRT changing `List<ConnectionObserver>` 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 <gbad...@redhat.com> wrote: > In 4.2 and 4.3, it was possible to add a ConnectionObserver > using > org.hibernate.engine.jdbc.spi.LogicalConnectionImplementor#addObserver(ConnectionObserver > observer). > > Starting in 5.0, LogicalConnectionImplementor was moved > to org.hibernate.resource.jdbc.spi, and it's no longer possible to add a > ConnectionObserver. > > For now, I think it's OK to change JdbcObserverImpl#observers from a > List<ConnectionObserver>, to: > > private final transient ConnectionObserver observer; > > Also, I'm fine with ConnectionObserver being deprecated, and later removed. > > Steve, WDYT? > > > On Tue, Aug 20, 2019 at 10:55 AM Sanne Grinovero <sa...@hibernate.org> > wrote: > > > 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 questions: > > > > # Is this expected? > > > > # Should we deprecate the SPI so to make this more explict in the > > future, and possibly eventually discuss removing it? > > > > # Since I'm working on performance optimisations, may I take advantage > > currently of the fact that there's only one registered? We're > > currently iterating "the list of obeservers"... > > > > Thanks > > _______________________________________________ > > 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 > _______________________________________________ hibernate-dev mailing list hibernate-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/hibernate-dev
