Fw: PATCH (Re: Startup message issues)

Sorry.  Again forgot to copy the list.  (I'll get the hang of this
eventually...).  Here's what I mistakenly sent only to Kris, but meant to send
to the list.

Chris Smith wrote:
> Kris Jurka wrote:
> > This would be a good thing to do.  I believe the V3 protocol was put
> > in with the goal of getting it working which meant a copy, paste,
> > and minimal edit.  So yes V3 implies 7.4 and it would be nice if it
> > took full advantage of what is offered.
>
> Okay, great!  Could someone review the attached patch?  It makes the
> following changes:
>
>     1. Remove backward-compatibility for pre-7.4 versions from
>        the v3 protocol connection code.
>     2. Move client_encoding and DateStyle setup to startup message.
>     3. Parse ParameterStatus for server_version.
>
> The result of #2 and #3 is that v3 protocol connections are
> established with only one round-trip to the server (in the trust auth
> case; two when authentication is required).  I've done some ad-hoc
> testing... is there a test suite that should be run against this?

--
www.designacourse.com
The Easiest Way to Train Anyone... Anywhere.

Chris Smith - Lead Software Developer/Technical Trainer
MindIQ Corporation