Horton

Re: [BUG] temporary file usage report with extended protocol and unnamed portals

Frédéric Yhuel <frederic.yhuel@dalibo.com>

View thread View on postgr.es
From: Frédéric Yhuel <frederic.yhuel@dalibo.com>
To: Sami Imseih <samimseih@gmail.com>, Michael Paquier <michael@paquier.xyz>
Cc: Mircea Cadariu <cadariu.mircea@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Benoit Lobréau <benoit.lobreau@dalibo.com>, Guillaume Lelarge <guillaume.lelarge@dalibo.com>, Pierrick Chovelon <pierrick.chovelon@dalibo.com>
Date: 2025-09-19T07:06:49Z
Lists: pgsql-hackers 

On 9/18/25 18:06, Sami Imseih wrote:
> I also realized that the initial tests we've been working with included:
> 
> ```
> +log_statement = all
> +log_min_duration_statement = 0
> ```
> 

Yes, you added these two in your v8, together with a useless 
log_line_prefix.

> which caused additional noise and did not isolate the temp logging,
> and that could mislead the tests.

I agree.

Commits

  1. Drop unnamed portal immediately after execution to completion

  2. Override log_error_verbosity to "default" in test 009_log_temp_files

  3. Add tests for logging of temporary file removal and statement