On Wed, Nov 5, 2025 at 12:43 AM Michael Paquier <[email protected]> wrote: > Drop unnamed portal immediately after execution to completion
This patch doesn't look well-considered to me. One problem is that it's a wire protocol change to fix a minor logging anomaly, which seems disproportionate. Another problem is that the new portal-drop behavior is conditional on whether XACT_FLAGS_NEEDIMMEDIATECOMMIT gets set, which seems unprincipled. In addition to those points, I am not entirely certain that the "here is no need for it beyond this point" comment is correct. I mean, I think it will normally be true, but what if the client wants to send a Describe message after-the-fact, or an additional Execute message that will presumably return zero rows? -- Robert Haas EDB: http://www.enterprisedb.com
