Re: psql not responding to SIGINT upon db reconnection
Tristan Partin <tristan@neon.tech>
From: "Tristan Partin" <tristan@neon.tech>
To: "Jelte Fennema-Nio" <postgres@jeltef.nl>
Cc: "Robert Haas" <robertmhaas@gmail.com>, "Heikki Linnakangas"
<hlinnaka@iki.fi>, <pgsql-hackers@postgresql.org>, "Shlok Kyal"
<shlok.kyal.oss@gmail.com>
Date: 2024-04-03T14:55:08Z
Lists: pgsql-hackers
On Wed Apr 3, 2024 at 9:50 AM CDT, Jelte Fennema-Nio wrote: > On Wed, 3 Apr 2024 at 16:31, Tom Lane <tgl@sss.pgh.pa.us> wrote: > > If we do the latter, we will almost certainly get pushback from > > distros who check for library ABI breaks. I fear the comment > > suggesting that we could remove it someday is too optimistic. > > Alright, changed patch 0002 to keep the variant. But remove it from > the recently added switch/case. And also updated the comment to remove > the "for awhile". Removing from the switch statement causes a warning: > [1/2] Compiling C object src/bin/psql/psql.p/command.c.o > ../src/bin/psql/command.c: In function ‘wait_until_connected’: > ../src/bin/psql/command.c:3803:17: warning: enumeration value ‘PGRES_POLLING_ACTIVE’ not handled in switch [-Wswitch] > 3803 | switch (PQconnectPoll(conn)) > | ^~~~~~ -- Tristan Partin Neon (https://neon.tech)
Commits
Same data as JSON:
GET /api/v1/messages/:b64id/commits
the thread's linked commits as JSON, with link sources.
API reference →
-
Fix old, misleading comment for PGRES_POLLING_ACTIVE.
- e57fe3824ee7 17.0 landed
-
Remove reachable call to pg_unreachable().
- 12b964d7815b 17.0 landed
-
Allow SIGINT to cancel psql database reconnections.
- cafe1056558f 17.0 landed
-
Expose PQsocketPoll via libpq
- f5e4dedfa81f 17.0 landed
-
Fix query cancellation handling in psql
- 5d43c3c54d77 13.0 cited