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