Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary files - Mailing list pgsql-hackers

From Guillaume Smet
Subject Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary files
Date
Msg-id 1d4e0c10701121123v7dbac21fqe298046b05c35896@mail.gmail.com
Whole thread Raw
In response to Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary  (Bruce Momjian <bruce@momjian.us>)
Responses Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary files  (Tom Lane <tgl@sss.pgh.pa.us>)
Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary  (Bruce Momjian <bruce@momjian.us>)
List pgsql-hackers
Hi Bruce,

Thanks for your answer.

On 1/12/07, Bruce Momjian <bruce@momjian.us> wrote:
> We have the ability to conditionally print statements based on error
> level, but LOG isn't a valid level for log_min_error_statement.
>
> We could add a parameter that few people would use, but the right way to
> do this is to log all queries.
>
> We do not allow unlimited logging control or the system would be too
> complex.

That's not what I had in mind. I was asking if the text of the query
was available when logging the temp file usage. If so it could be good
to add a DETAIL line with it directly and systematically when logging
the temp file usage.

Sure that if you log every query, you could find which query was
responsible for that  temp file but I suspect that this new log
feature will be used on production servers and so usually without
statement logging enabled.

IMHO, it's really important to know which queries are responsible for
the temp file usage.

If the text of the query is not available when logging the temp file
usage then I agree that we cannot do anything.

--
Guillaume

pgsql-hackers by date:

Previous
From: Bruce Momjian
Date:
Subject: Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary
Next
From: Tom Lane
Date:
Subject: Re: [pgsql-patches] [PATCHES] Patch to log usage of temporary files