Horton

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

Michael Paquier <michael@paquier.xyz>

View thread View on postgr.es
From: Michael Paquier <michael@paquier.xyz>
To: Sami Imseih <samimseih@gmail.com>
Cc: Mircea Cadariu <cadariu.mircea@gmail.com>, Guillaume Lelarge <guillaume.lelarge@dalibo.com>, Frédéric Yhuel <frederic.yhuel@dalibo.com>, Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, Benoit Lobréau <benoit.lobreau@dalibo.com>, Pierrick Chovelon <pierrick.chovelon@dalibo.com>
Date: 2025-10-16T23:23:48Z
Lists: pgsql-hackers 
On Thu, Oct 16, 2025 at 04:04:41PM -0500, Sami Imseih wrote:
>> prion has just failed because it uses log_error_verbosity = verbose:
>> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=prion&dt=2025-10-16%2001%3A23%3A06
>>
>> Thanks for progressing this! In v15 attached, I adjusted the regex to work with all settings mentioned in the thread so far.
>>
>> How to check if there's any other setting like this to cover?
> 
> It looks like this was resolved with 02c171f, which overrides the
> log_error_verbosity to 'default' in 009_log_temp_files.pl
> 
> https://buildfarm.postgresql.org/cgi-bin/show_history.pl?nm=prion&br=master

Yes, and the other members are not complaining as far as I can see, so
we should be clear in terms of the tests.  Now, back to the main
issue..
--
Michael

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