Hi,
On 20/04/2025 00:42, Sami Imseih wrote:
> [...]
>> I'm frankly inclined to do nothing, but if we must do something,
>> the way to fix it here would be to transiently set debug_query_string
>> to NULL so that the actions of PortalDrop aren't blamed on any
>> particular query.
>
> I think this is better, because I rather we avoid lines like the below in which
> there are temp files being reported all with STATEMENT logging
> of a different query. It's better to just not show STATEMENT at all.
>
We should definitely avoid blaming the wrong query. I'd prefer we always
log queries using temp files but if this isn't doable easily or without
mistakes, then don't log the query at all.
--
Guillaume Lelarge
Consultant
https://dalibo.com