[PATCH]Re: Which version of driver to use ?

From: Csaba Nagy <nagy(at)ecircle-ag(dot)com>
To: Dave Cramer <davec(at)fastcrypt(dot)com>
Cc: Postgres JDBC <pgsql-jdbc(at)postgresql(dot)org>
Subject: [PATCH]Re: Which version of driver to use ?
Date: 2005-06-27 12:06:18
Message-ID: 1119873978.3994.126.camel@coppola.muc.ecircle.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-jdbc

If it is of interest to anyone,

I have back-patched the PooledConnectionImpl in the JDBC driver coming
with the 7.4 postgres server to properly handle connection problems. The
code is taken from the latest 8.1 dev source with the necessary
adaptation.
Diff file is attached, was taken against CVS branch REL7_4_STABLE of the
postgres server source.

Cheers,
Csaba.

On Fri, 2005-06-24 at 18:28, Csaba Nagy wrote:
> Thanks Dave,
>
> The connection URL version will be good I guess, I'll try tomorrow.
>
> Cheers,
> Csaba.
>
> On Fri, 2005-06-24 at 18:25, Dave Cramer wrote:
> > Csaba,
> >
> > Yeah, you can force v2 protocol on the 8.0 driver
> >
> > just add ?protocolVersion=2 to the connection url
> >
> > see
> > http://jdbc.postgresql.org/documentation/80/connect.html#connection-
> > parameters
> >
> > for more details
> >
> > re the 74 code : it's in the main server tree for version 7.4 tag is
> > REL7_4_STABLE
> >
> > Dave
> >
> > On 24-Jun-05, at 11:33 AM, Csaba Nagy wrote:
> >
> > > Hi all,
> > >
> > > We have a postgres 7.4 installation with production code running on
> > > it.
> > > The code is written in Java and uses the JDBC driver to access it.
> > >
> > > I recently discovered that the JDBC driver we were using is not
> > > properly
> > > detecting connection problems in the pooled connection
> > > implementation to
> > > report them to our connection pool. However, when giving a go to fix
> > > this, I also discovered that the latest 8.0 driver fixes this problem
> > > (I've checked the code).
> > >
> > > So I've tried to upgrade to the 8.0 driver, but failed: our code is
> > > pretty sloppy with the parameter types of prepared statements, and the
> > > new driver is very strict about them. Eventually we will fix this, but
> > > for the deployed code it's not an option.
> > >
> > > Question: is there a way to make the 8.0 driver less strict ? E.g. we
> > > have quite a few places where numeric fields are set with "setString",
> > > and other things like this.
> > >
> > > Alternatively, have anybody back-patched the connection error
> > > notification fix to the 7.4 driver line ? If not, I could do that
> > > myself, in this case what's the recommended way of getting the latest
> > > 7.4 sources ? Is there actually a CVS branch for it for bug-fixes ?
> > >
> > > Thanks,
> > > Csaba.
> > >
> > >
> > >
> > > ---------------------------(end of
> > > broadcast)---------------------------
> > > TIP 4: Don't 'kill -9' the postmaster
> > >
> > >
> >
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 3: if posting/reading through Usenet, please send an appropriate
> subscribe-nomail command to majordomo(at)postgresql(dot)org so that your
> message can get through to the mailing list cleanly

Attachment Content-Type Size
PooledConnectionImpl.java.diff text/x-patch 7.1 KB

In response to

Browse pgsql-jdbc by date

  From Date Subject
Next Message Enrique Ibarra 2005-06-27 18:22:11 Re: Problems loading the driver
Previous Message sarves 2005-06-27 08:54:42 compatibility issue