Re: FW: REVIEW: Allow formatting in log_line_prefix - Mailing list pgsql-hackers

From Robert Haas
Subject Re: FW: REVIEW: Allow formatting in log_line_prefix
Date
Msg-id CA+TgmoZycK-ko7_FqYU+z7Ysj=9j9LvzxLK5D+yvqg0tnkzErg@mail.gmail.com
Whole thread Raw
In response to Re: FW: REVIEW: Allow formatting in log_line_prefix  (David Rowley <dgrowleyml@gmail.com>)
Responses Re: FW: REVIEW: Allow formatting in log_line_prefix
List pgsql-hackers
On Thu, Sep 26, 2013 at 3:55 PM, David Rowley <dgrowleyml@gmail.com> wrote:
> I think I must have forgot to save it before I emailed it...
>
> Here's the correct version.

Ah ha.  Looks better.

I'm working on getting this committed now.  Aside from some stylistic
things, I've found one serious functional bug, which is that you need
to test padding != 0, not padding > 0, when deciding which path to
take.  No need to send a new patch, I've already fixed it in my local
copy...

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company



pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: INSERT...ON DUPLICATE KEY LOCK FOR UPDATE
Next
From: Bruce Momjian
Date:
Subject: Re: record identical operator - Review