Remove assertion from PortalRunMulti
Álvaro Herrera <alvherre@kurilemu.de>
Remove assertion from PortalRunMulti We have an assertion to ensure that a command tag has been assigned by the time we're done executing, but if we happen to execute a command with no queries, the assertion would fail. Per discussion, rather than contort things to get a tag assigned, just remove the assertion. Oversight in 2f9661311b83. That commit also retained a comment that explained logic that had been adjacent to it but diffused into various places, leaving none apt to keep part of the comment. Remove that part, and rewrite what remains for extra clarity. Bug: #18984 Backpatch-through: 13 Reported-by: Aleksander Alekseev <aleksander@tigerdata.com> Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us> Reviewed-by: Michaël Paquier <michael@paquier.xyz> Discussion: https://postgr.es/m/18984-0f4778a6599ac3ae@postgresql.org
Files
| Path | Change | +/− |
|---|---|---|
| src/backend/tcop/pquery.c | modified | +8 −17 |
Discussion
- BUG #18984: Empty prepared statement from psql \parse triggers assert in PortalRunMulti 21 messages · 2025-07-14 → 2025-07-17