Re: Allow cluster_name in log_line_prefix

2019-11-10 Thread Craig Ringer
On Sun, 3 Nov 2019 at 07:22, Vik Fearing wrote: > On 31/10/2019 08:47, Fujii Masao wrote: > > On Mon, Oct 28, 2019 at 1:33 PM Craig Ringer > wrote: > >> Hi folks > >> > >> I was recently surprised to notice that log_line_prefix doesn't support > a cluster_name placeholder. I suggest adding one.

Re: Allow cluster_name in log_line_prefix

2019-11-02 Thread Vik Fearing
On 31/10/2019 08:47, Fujii Masao wrote: > On Mon, Oct 28, 2019 at 1:33 PM Craig Ringer wrote: >> Hi folks >> >> I was recently surprised to notice that log_line_prefix doesn't support a >> cluster_name placeholder. I suggest adding one. If I don't hear objections >> I'll send a patch. > If we do

Re: Allow cluster_name in log_line_prefix

2019-10-31 Thread Andres Freund
Hi, On 2019-10-28 12:33:00 +0800, Craig Ringer wrote: > I was recently surprised to notice that log_line_prefix doesn't support a > cluster_name placeholder. I suggest adding one. If I don't hear objections > I'll send a patch. > > Before anyone asks "but why?!": > > * A constant (short) string

Re: Allow cluster_name in log_line_prefix

2019-10-31 Thread Fujii Masao
On Mon, Oct 28, 2019 at 1:33 PM Craig Ringer wrote: > > Hi folks > > I was recently surprised to notice that log_line_prefix doesn't support a > cluster_name placeholder. I suggest adding one. If I don't hear objections > I'll send a patch. If we do this, cluster_name should be included in csvl

Re: Allow cluster_name in log_line_prefix

2019-10-30 Thread Tatsuo Ishii
> Hi folks > > I was recently surprised to notice that log_line_prefix doesn't support a > cluster_name placeholder. I suggest adding one. If I don't hear objections > I'll send a patch. I think it'd be a good thing for users. Best regards, -- Tatsuo Ishii SRA OSS, Inc. Japan English: http://www

Re: Allow cluster_name in log_line_prefix

2019-10-30 Thread Thomas Munro
On Mon, Oct 28, 2019 at 3:33 PM Craig Ringer wrote: > I was recently surprised to notice that log_line_prefix doesn't support a > cluster_name placeholder. I suggest adding one. If I don't hear objections > I'll send a patch. +1

Allow cluster_name in log_line_prefix

2019-10-27 Thread Craig Ringer
Hi folks I was recently surprised to notice that log_line_prefix doesn't support a cluster_name placeholder. I suggest adding one. If I don't hear objections I'll send a patch. Before anyone asks "but why?!": * A constant (short) string in log_line_prefix is immensely useful when working with lo