Re: Add support for logging the current role

Stephen Frost <sfrost@snowman.net>

From: Stephen Frost <sfrost@snowman.net>

To: Robert Haas <robertmhaas@gmail.com>

Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>

Date: 2011-01-12T15:12:23Z

Lists: pgsql-hackers

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Add support for an application_name parameter, which is displayed in
- 8217cfbd9918 9.0.0 cited
Make CSV column ordering a bit more logical.
- 230e8962f3a4 8.3.0 cited
Extend the format of CSV logs to include the additional information supplied
- 3bf66d6f1c3a 8.3.0 cited
Add virtual transaction IDs to CSVLOG output, so that messages coming from
- 77c166ba6cf6 8.3.0 cited
Provide for logfiles in machine readable CSV format. In consequence, rename
- fd801f4faa8e 8.3.0 cited

* Robert Haas (robertmhaas@gmail.com) wrote:
> On Wed, Jan 12, 2011 at 9:23 AM, Stephen Frost <sfrost@snowman.net> wrote:
> > Minor enhancement, but a valuable one imv.  Hopefully there aren't any
> > issues with it. :)
> 
> 1. Why %o?  That's not obviously mnemonic.  Perhaps %U?

r was taken? :)  I'm not sure I like %U, but in the end I don't *really*
care.  I'll update it to %U and wait for someone else to complain.

> 2. It won't be clear to people reading this what the difference is
> between %u and this.  You probably need to reword the documentation
> for the existing option as well as documenting the new one.

Fair enough.

> 3. Please attach the patch rather than including it inline, if possible.

Hrm, I could have sworn that Tom had asked for the exact opposite in the
past, but either way is fine by me.

	Stephen