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
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 |
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 |