Re: [ADMIN] 9.5 new setting "cluster name" and logging

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
Cc: Joe Conway <mail(at)joeconway(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Evan Rempel <erempel(at)uvic(dot)ca>, pgsql-admin(at)postgresql(dot)org, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [ADMIN] 9.5 new setting "cluster name" and logging
Date: 2016-02-08 20:42:06
Message-ID: 20160208204206.GV3331@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-admin pgsql-hackers

Thomas,

* Thomas Munro (thomas(dot)munro(at)enterprisedb(dot)com) wrote:
> On Tue, Feb 9, 2016 at 5:30 AM, Joe Conway <mail(at)joeconway(dot)com> wrote:
> > On 02/08/2016 06:24 AM, Andres Freund wrote:
> >> On 2016-01-29 22:19:45 -0800, Evan Rempel wrote:
> >>> Now that there is a setting to give a cluster a "name", it would be nice to
> >>> have an escape sequence in the log_line_prefix setting that could reference
> >>> the cluster_name.
> >>
> >> I've argued[1][2] for this when cluster_name was introduced, but back
> >> then I seemed to have been the only one arguing for it. Josh later
> >> jumped that train.
> >>
> >> Given that we now had a number of people wishing for this, can we maybe
> >> reconsider?
> >
> > Seems like a reasonable idea to me. But if we add a log_line_prefix
> > setting, shouldn't we also add it to csvlog output too?
>
> Here's a tiny patch adding support for %C to log_line_prefix (this was
> part of the cluster_name patch that didn't go it).
>
> Given that csvlog's output format is hardcoded in write_csvlog, how is
> it supposed to evolve without upsetting consumers of this data?
> Wouldn't we first need to add a GUC that lets you control the columns
> it outputs?

Not sure if you really want to go there, but I do agree with you and
there's a thread from a few years back about something similar:

http://www.postgresql.org/message-id/flat/20110112142345(dot)GA4933(at)tamriel(dot)snowman(dot)net

Included in that thread is a patch, which likely requires some dusting
off, to add exactly that ability.

Thanks!

Stephen

In response to

Browse pgsql-admin by date

  From Date Subject
Next Message Yogi Barot 2016-02-09 18:46:33 Re: pg_upgrade error
Previous Message Thomas Munro 2016-02-08 20:36:17 Re: [ADMIN] 9.5 new setting "cluster name" and logging

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2016-02-08 20:45:28 Re: Tracing down buildfarm "postmaster does not shut down" failures
Previous Message Alvaro Herrera 2016-02-08 20:41:23 Re: Tracing down buildfarm "postmaster does not shut down" failures