Re: Patch to remove connection hook and JDK 1.3 dependencies - Mailing list pgsql-jdbc

From Joseph Shraibman
Subject Re: Patch to remove connection hook and JDK 1.3 dependencies
Date
Msg-id 3B4A36B9.EBC8A057@selectacast.net
Whole thread Raw
In response to Re: Re: Patch to remove connection hook and JDK 1.3 dependencies  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-jdbc
Joseph Shraibman wrote:
>
<snip>

OK I looked at ConnectionHook in the cvs and there is obviously code
there to close the connections, but where is the code that calls
ConnectionHook.open()?  That's probably the problem.

Also a Vector should be used instead of an ArrayList because there could
be multiple threads accessing the jdbc driver at the same time.  It
could be that open() and close() are only supposed to be called from
code that is locked anyway but I can't verify that.

Also I'm wondering why there is a ConnectionHook class at all.  It seems
to me it would be better to have this stuff in Connection itself and we
wouldn't have to do that strange workaround of calling init().

If I can connect to the cvs I'll try and code it.


--
Joseph Shraibman
jks@selectacast.net
Increase signal to noise ratio.  http://www.targabot.com

pgsql-jdbc by date:

Previous
From: Barry Lind
Date:
Subject: Re: [PATCH] Cleanup of JDBC character encoding
Next
From: Joseph Shraibman
Date:
Subject: Shutdown hook imp