From: | Vik Fearing <vik(dot)fearing(at)dalibo(dot)com> |
---|---|
To: | Andres Freund <andres(at)2ndquadrant(dot)com>, Thomas Munro <munro(at)ip9(dot)org> |
Cc: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Abhijit Menon-Sen <ams(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Stephen Frost <sfrost(at)snowman(dot)net>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Cluster name in ps output |
Date: | 2014-07-01 18:45:25 |
Message-ID: | 53B301C5.2040800@dalibo.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 06/29/2014 02:25 PM, Andres Freund wrote:
> On 2014-06-29 11:11:14 +0100, Thomas Munro wrote:
>> > On 29 June 2014 10:55, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>>> > > So, I'd looked at it with an eye towards committing it and found some
>>> > > more things. I've now
>>> > > * added the restriction that the cluster name can only be ASCII. It's
>>> > > shown from several clusters with differing encodings, and it's shown
>>> > > in process titles, so that seems wise.
>>> > > * moved everything to the LOGGING_WHAT category
>>> > > * added minimal documention to monitoring.sgml
>>> > > * expanded the config.sgml entry to mention the restriction on the name.
>>> > > * Changed the GUCs short description
>> >
>> > Thank you.
>> >
>>> > > I also think, but haven't done so, we should add a double colon after
>>> > > the cluster name, so it's not:
>>> > >
>>> > > postgres: server1 stats collector process
>>> > > but
>>> > > postgres: server1: stats collector process
>> >
>> > +1
>
> Committed with the above changes. Thanks for the contribution!
Is there a reason for not using this in synchronous_standby_names,
perhaps falling back to application_name if not set?
--
Vik
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2014-07-01 18:52:00 | Re: Spinlocks and compiler/memory barriers |
Previous Message | Robert Haas | 2014-07-01 18:02:02 | Re: buildfarm and "rolling release" distros |