| From: | Sami Imseih <samimseih(at)gmail(dot)com> |
|---|---|
| To: | Mircea Cadariu <cadariu(dot)mircea(at)gmail(dot)com> |
| Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Frédéric Yhuel <frederic(dot)yhuel(at)dalibo(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Benoit Lobréau <benoit(dot)lobreau(at)dalibo(dot)com>, Guillaume Lelarge <guillaume(dot)lelarge(at)dalibo(dot)com>, Pierrick Chovelon <pierrick(dot)chovelon(at)dalibo(dot)com> |
| Subject: | Re: [BUG] temporary file usage report with extended protocol and unnamed portals |
| Date: | 2025-09-19 12:50:09 |
| Message-ID: | CAA5RZ0tT8bqt-DJajGFx_uTYhcXJLWP1JW4VBAikRufCAfChRA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> The "everything" also includes the query_text.
>
> This makes me wonder then if all it takes is just adding this to PortalDrop (proposed earlier in the thread by Frédéric):
One thing I did not like about that approach is that we will need to
save the current debug_query_string inside PortalDrop before
temporarily setting it to the one from the about to be dropped
portal, and then set it back to the saved one before exiting.
Otherwise, we might end up logging the wrong query in some cases
(although I could not find a test case that proves my worry).
With v12, drop_unnamed_portal sets the debug_query_string of the
portal, and we know right after drop_unnamed_portal is completed
the debug_query_string is set to the current query.
--
Sami
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2025-09-19 12:51:55 | Re: MinGW cross compilation failure on Debian trixie |
| Previous Message | Álvaro Herrera | 2025-09-19 12:49:02 | Re: REPACK and naming |