From: | Dave Cramer <pg(at)fastcrypt(dot)com> |
---|---|
To: | Vitalii Tymchyshyn <vit(at)tym(dot)im> |
Cc: | Tom Dunstan <pgsql(at)tomd(dot)cc>, PG-JDBC Mailing List <pgsql-jdbc(at)postgresql(dot)org>, "Heiko W(dot) Rupp" <hwr(at)pilhuhn(dot)de> |
Subject: | Re: Remove usage of finalizers ? |
Date: | 2013-10-23 01:03:57 |
Message-ID: | CADK3HH+mydhDZr+B3ER61MYBdwfoXT7WYKNnObRJZBFRya4rBg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-jdbc |
That's rather harsh. How do you see this as introducing a server side leak
? Statements are supposed to be closed by applications as are connections.
Dave Cramer
dave.cramer(at)credativ(dot)ca
http://www.credativ.ca
On Tue, Oct 22, 2013 at 8:36 PM, Vitalii Tymchyshyn <vit(at)tym(dot)im> wrote:
> 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(at)tomd(dot)cc> написав:
>
> On 21 October 2013 23:03, Dave Cramer <pg(at)fastcrypt(dot)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
>>
>
From | Date | Subject | |
---|---|---|---|
Next Message | David Wall | 2013-10-23 01:19:28 | Re: Remove usage of finalizers ? |
Previous Message | Vitalii Tymchyshyn | 2013-10-23 00:36:25 | Re: Remove usage of finalizers ? |