Re: [PATCHES] WIP patch - INSERT-able log statements

From: "Guillaume Smet" <guillaume(dot)smet(at)gmail(dot)com>
To: "Guillaume Smet" <guillaume(dot)smet(at)gmail(dot)com>, "Greg Smith" <gsmith(at)gregsmith(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCHES] WIP patch - INSERT-able log statements
Date: 2007-02-19 17:31:53
Message-ID: 1d4e0c10702190931v9da1786ye55b37b52e317be2@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

On 2/19/07, Alvaro Herrera <alvherre(at)commandprompt(dot)com> wrote:
> Guillaume Smet escribió:
> > On 2/19/07, Alvaro Herrera <alvherre(at)commandprompt(dot)com> wrote:
> > >So add the session ID (%c) to log_line_prefix.
> >
> > It could work if log_line_prefix was added before every line but it's
> > definitely not the case:
> > myuser mydb 45d9d615.4abe LOG: duration : 185.223 ms, statement : SELECT *
> > FROM lieu
> > LIMIT 10;
> > if you execute:
> > SELECT *
> > FROM lieu
> > LIMIT 10;
>
> Interesting. I wonder why didn't you report this as a bug before?
> Maybe we could have discussed it and fixed it.

Perhaps because I thought it was not really a bug but the intended behaviour.
Syslog has the same behaviour and it's quite logical when you consider
how queries are logged (I've spent a few hours in the logging code).
Syslog has exactly the same behaviour but adds the necessary context
information.
I'm pretty sure I have explained the problem a few times on the lists
though but perhaps it was just on IRC.

From the feedback I have on pgFouine, very few people think it's a
real problem, probably because they don't use query logging as we do:
our production servers have it enabled all the time and we have a high
load on them so this particular case is a common case for us.

(Second try to move this discussion to -hackers)

--
Guillaume

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2007-02-19 17:51:42 Re: [PATCHES] WIP patch - INSERT-able log statements
Previous Message Florian G. Pflug 2007-02-19 17:16:31 Re: Confusing message on startup after a crash while recovering

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2007-02-19 17:44:28 Re: [BUGS] BUG #2977: dow doesn't conform to ISO-8601
Previous Message Alvaro Herrera 2007-02-19 17:15:55 Re: WIP patch - INSERT-able log statements