Re: add support for logging current role (what to review?) - Mailing list pgsql-hackers

From Alex Hunsaker
Subject Re: add support for logging current role (what to review?)
Date
Msg-id BANLkTi=P3KCwyTgCbG4C+GSWn7iSFpgqiA@mail.gmail.com
Whole thread Raw
In response to Re: add support for logging current role (what to review?)  (Stephen Frost <sfrost@snowman.net>)
List pgsql-hackers
On Thu, Jun 30, 2011 at 16:35, Stephen Frost <sfrost@snowman.net> wrote:

> I do want to rework the logging infrastructure (as discussed in the dev
> meeting), but I see that whole thing as rather orthogonal to this
> change.

*Shrug* Fine by me, im not going to argue that you should or shouldn't
rework the logging infrastructure. There seem to be people on both
sides of the fence... Including me. Im of the opinion that for *this*
field we should just add it, I think it stands on its own legs. For
some of the other columns (duration, pl line # etc)... well It might
make more sense to have a knob.

So uhh.. Im still confused at exactly what patch I should be looking
at because of my opinion that we should just add this field.

Stephen (others?), whats your preference? Maybe this should be two
patches (one that adds the field, another that adds a header)?


pgsql-hackers by date:

Previous
From: "Jaime Casanova"
Date:
Subject: Re: add support for logging current role (what to review?)
Next
From: Josh Berkus
Date:
Subject: Re: beta3?