Re: Remove usage of finalizers ? - Mailing list pgsql-jdbc

From Vitalii Tymchyshyn
Subject Re: Remove usage of finalizers ?
Date
Msg-id CABWW-d1iMgM9-fs2uJ2Vk5rTQnN29nvUvfhQUg0k=jxao-T_KA@mail.gmail.com
Whole thread Raw
In response to Re: Remove usage of finalizers ?  (Tom Dunstan <pgsql@tomd.cc>)
Responses Re: Remove usage of finalizers ?  (Dave Cramer <pg@fastcrypt.com>)
Re: Remove usage of finalizers ?  (Tom Dunstan <pgsql@tomd.cc>)
Re: Remove usage of finalizers ?  ("Heiko W. Rupp" <hwr@pilhuhn.de>)
List pgsql-jdbc

As for me, introducing server-side leak would be plain wrong. If it will be done, please announce in the list, I will stop recommending using postgresql in java projects.

23 жовт. 2013 03:12, користувач "Tom Dunstan" <pgsql@tomd.cc> написав:
On 21 October 2013 23:03, Dave Cramer <pg@fastcrypt.com> wrote:
> Please send patches for possible solutions. I think I would be OK with
> removing them and letting people find their Statement leaks

One option if we want to keep the existing behaviour available in some
way would be to remove the finalize method from AbstractJdbc2Statement
and create subclasses of the various concrete statement classes which
could be used when a debug flag is switched on. e.g.
DebugJdbc3Statement, DebugJdbc3PreparedStatement,
DebugJdbc3CallableStatement etc which have the finalizer present.

There would be quite a few of them though, and we'd need to have a
switch everywhere that one of those is instantiated, or introduce a
factory. It's a bit gross.

IMO tracing non-closed statement leaks is probably better done in a
connection pool or jdbc debugging library anyway, so maybe we
shouldn't bother and just remove it.

One question is this, though: how many users out there aren't closing
their statements currently and are relying, accidentally or
deliberately, on the current behaviour?

Tom

pgsql-jdbc by date:

Previous
From: Tom Dunstan
Date:
Subject: Re: Remove usage of finalizers ?
Next
From: Dave Cramer
Date:
Subject: Re: Remove usage of finalizers ?