From: | Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Fabien COELHO <coelho(at)cri(dot)ensmp(dot)fr>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Make psql's qeury canceling test simple by using signal() routine of IPC::Run |
Date: | 2023-09-13 03:58:13 |
Message-ID: | 20230913125813.1211700ee783324e39593c72@sraoss.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, 13 Sep 2023 10:20:23 +0900
Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> On Tue, Sep 12, 2023 at 03:18:05PM +0900, Michael Paquier wrote:
> > On Wed, Sep 06, 2023 at 12:45:24AM +0900, Yugo NAGATA wrote:
> > > I attached the update patch. I removed the incorrect comments and
> > > unnecessary lines. Also, I rewrote the test to use "skip_all" instead
> > > of SKIP because we skip the whole test rather than a part of it.
> >
> > Thanks for checking how IPC::Run behaves in this case on Windows!
> >
> > Right. This test is currently setting up a node for nothing, so let's
> > skip this test entirely under $windows_os and move on. I'll backpatch
> > that down to 15 once the embargo on REL_16_STABLE is lifted with the
> > 16.0 tag.
>
> At the end, I have split this change into two:
> - One to disable the test to run on Windows, skipping the wasted node
> initialization, and applied that down to 15.
> - One to switch to signal(), only for HEAD to see what happens in the
> buildfarm once the test is able to run on platforms that do not
> support PPID. I am wondering as well how IPC::Run::signal is stable,
> as it is the first time we would use it, AFAIK.
Thank you for pushing them.
I agree with the suspection about IPC::Run::singnal, so I'll also
check the buildfarm result.
Regards,
Yugo Nagata
> --
> Michael
--
Yugo NAGATA <nagata(at)sraoss(dot)co(dot)jp>
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2023-09-13 04:14:12 | Re: Query execution in Perl TAP tests needs work |
Previous Message | Lepikhov Andrei | 2023-09-13 03:48:43 | Re: POC: GUC option for skipping shared buffers in core dumps |