Re: Removing support for v1 and v2 protocols?

From: Craig Ringer <craig(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: "pgsql-odbc(at)postgresql(dot)org" <pgsql-odbc(at)postgresql(dot)org>
Subject: Re: Removing support for v1 and v2 protocols?
Date: 2014-06-19 03:49:00
Message-ID: 53A25DAC.2080007@2ndquadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-odbc

On 06/19/2014 11:10 AM, Tom Lane wrote:
> Craig Ringer <craig(at)2ndquadrant(dot)com> writes:
>> I've just found a bug (followup mail to come) that seems to be tied up
>> in support for the old v1/v2 protocols.
>
>> Is there any reason to keep all that cruft? v3 was introduced in 7.4, in
>> 2003-11-17 .
>
> FWIW, I've been thinking of proposing that we drop v2 protocol support
> in libpq and the backend.

Sounds great, and we could cut a bunch of code out of PgJDBC then.
That'll make Dave Cramer happy.

> I don't recall at the moment what the differences were between v1 and v2,
> but I think that libpq hasn't had v1 support since 7.4, if not further
> back. The backend might still nominally support v1, but that code is
> surely pretty untested anymore. So you definitely don't want v1, if
> indeed ODBC still contains support for that.

It appears to, in that it has places where it tests for whether it's
talking to 6.4+ (!) or not. It might just be scattered remenants, but
that's almost worse.

*Begins sharpening razor*

I'm a bit lost as to why psqlODBC does so much of this its self, instead
of just using libpq. PgJDBC does it because it wants to be a JDBC Type 4
driver, i.e. all Java code, so it can't talk to libpq over JNI/JNA. That
concern does not apply to psqlODBC, which in fact already links to libpq
but doesn't seem to use it for protocol handling.

--
Craig Ringer http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-odbc by date

  From Date Subject
Next Message Michael Paquier 2014-06-19 03:49:09 Re: Removing support for v1 and v2 protocols?
Previous Message Craig Ringer 2014-06-19 03:44:21 Protocol de-synchronisation bug, bogus query sent