Re: Binary transfer patches v15

Mikko,

Great, I'll try this as soon as possible. I did have a problem with
running out of connections in the test. Did you ever see that ?

Dave Cramer

dave.cramer(at)credativ(dot)ca
http://www.credativ.ca

On Wed, Sep 21, 2011 at 6:22 PM, Mikko Tiihonen
<mikko(dot)tiihonen(at)nitorcreations(dot)com> wrote:
> I believe these are now ready for merging to cvs head. The patch set now
> passes all tests on jdk7u2 + postgres 9.1.0.
>
> Changes from last week:
> - Rebased to latest cvs
> - Fixed connection leak in jdbc3/DatabaseMetaDataTest.java
>   => now it is possible to run the unit tests with max_connections=2 in
> postgresql.conf
> - Fixed ForceBinaryTransfer bug in DatabaseEncodingTest
>   => Required a new field to Jdbc2ResultSet to keep the real statement alive
>      (Thanks to Oliver for the good analysis on the problem)
> - Fixed UpdateableResultTest.testUpdateStreams with binary transfers
>   => now ResultSet.refreshRow() informs lower level code that the results
> are for
>      updateable result set and should not use binary transfer
>
> Any comments on what still needs to be done before this is accepted?
>
> -Mikko
>
>
>
> --
> Sent via pgsql-jdbc mailing list (pgsql-jdbc(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-jdbc
>
>