Re: Deleting prepared statements from libpq.
Jelte Fennema <me@jeltef.nl>
From: Jelte Fennema <me@jeltef.nl>
To: Michael Paquier <michael@paquier.xyz>
Cc: jian he <jian.universality@gmail.com>,
Craig Ringer <craig@2ndquadrant.com>, Dmitry Igrishin <dmitigr@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@postgresql.org>, Daniele Varrazzo <daniele.varrazzo@gmail.com>
Date: 2023-06-19T09:44:52Z
Lists: pgsql-hackers
On Mon, 19 Jun 2023 at 01:57, Michael Paquier <michael@paquier.xyz> wrote: > +static int > +PQsendClose(PGconn *conn, char close_type, const char *close_target) > > Could it be better for this code path to issue an error if using a > non-supported close_type rather than sending it? Okay, you are > consistent with desc_type and PQsendDescribe(), just asking if it is > worth doing something about. Since it's not a publicly exposed function, I think it's fine as is. Seems easy enough to make sure libpq itself doesn't call it with unsupported arguments. And even if someone would do that accidentally, the server would report an error. > Perhaps there should be some tests for the two async routines, as > well? Done I also fixed all the typos/docs changes you and Jian mentioned, as well as improving some of the new docs myself.
Commits
-
libpq: Add support for Close on portals and statements
- 28b572656184 17.0 landed
-
Fix incorrect error message in libpq_pipeline
- 451ca5c1e699 14.9 landed
- 4fd633df5080 15.4 landed
- f69a7f08fd01 16.0 landed