Re: %d in log_line_prefix doesn't work for bg/autovacuum workers

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Andres Freund <andres(at)2ndquadrant(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: %d in log_line_prefix doesn't work for bg/autovacuum workers
Date: 2014-05-16 20:19:28
Message-ID: 22334.1400271568@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andres Freund <andres(at)2ndquadrant(dot)com> writes:
> On 2014-05-16 15:44:53 -0400, Tom Lane wrote:
>> But according to previous research, we don't
>> really guarantee that glibc thinks the encoding is what we think, anyway.
>> The commit messages for 54cd4f04576833abc394e131288bf3dd7dcf4806 and
>> ed437e2b27c48219a78f3504b0d05c17c2082d02 are relevant here. The second
>> one suggests that only "%.*s" is at risk not "%*s", but I'm not really
>> convinced right now. My recollection is that glibc will abandon
>> processing either format spec if it thinks the string is wrongly encoded.

> I've tested it here. From what it looks like it prints just fine but
> misjudges the width for padding.

Oh, good. We can live with that.

> You propose to revert?

Not if the data gets printed. I was afraid it wouldn't be.

(In any case, we could retain the feature and just do the padding
computations ourselves. But I now think we don't have to.)

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2014-05-16 20:30:59 Re: %d in log_line_prefix doesn't work for bg/autovacuum workers
Previous Message Andres Freund 2014-05-16 20:09:49 Re: %d in log_line_prefix doesn't work for bg/autovacuum workers