diff options
| author | Michael Paquier <michael@paquier.xyz> | 2023-09-13 10:10:04 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2023-09-13 10:10:04 +0900 |
| commit | 522a31ac873d7c6b02677f6d119c089a7dd09dd1 (patch) | |
| tree | 30e23da38998a814cdb1e513f6236776d3f28cae /src/backend/executor/nodeModifyTable.c | |
| parent | c53e288dba96bf42c5bfc21e75b45efbf37863d0 (diff) | |
| download | postgresql-522a31ac873d7c6b02677f6d119c089a7dd09dd1.tar.gz postgresql-522a31ac873d7c6b02677f6d119c089a7dd09dd1.zip | |
Switch psql's TAP test for query cancellation to use IPC::Run::signal()
Previously, the test relied on a trick with a shell to retrieve the PID
of the psql session to be stopped with SIGINT, that was skipped on
Windows. This commit changes the test to use IPC::Run::signal()
instead, which still does not work on Windows, but for a different
reason: SIGINT would stop the test before finishing.
This should allow the test to run on non-Windows platforms where PPID is
not supported (like NetBSD), spreading it a bit more across the
buildfarm. And the logic of the test is simpler.
It is the first time in the tree that IPC::Run::signal() is used, so, as
a matter of safety (or just call that as me having cold feet), no
backpatch is done, at least for now.
Author: Yugo NAGATA
Reviewed-by: Fabien Coelho
Discussion: https://postgr.es/m/20230810125935.22c2922ea5250ba79358965b@sraoss.co.jp
Diffstat (limited to 'src/backend/executor/nodeModifyTable.c')
0 files changed, 0 insertions, 0 deletions