On Wed, 2009-10-21 at 10:19 +0100, Dave Page wrote:
> On Wed, Oct 21, 2009 at 10:14 AM, Simon Riggs <simon@2ndquadrant.com> wrote:
>
> > ISTM much of the complexity discussed relates to this second feature.
> > Let's just concentrate on getting the connection-pool-identification
> > aspect of this done right and then maybe add second part later.
>
> That side of the patch works fine already (it's only a GUC after all),
> as does the during-connection option. The current problem is that old
> servers will barf if it's set in the startup packet.
>
> And yes, the stats part also works :-).
Then I say this feature is good to go. It's much needed. Well done.
Strip out the during-connection option for now. Let the startup packet
stuff come later, if ever.
-- Simon Riggs www.2ndQuadrant.com
