Re: Why is src/test/modules/committs/t/002_standby.pl flaky?
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Alexander Lakhin <exclusion@gmail.com>, Tom Lane <tgl@sss.pgh.pa.us>, Andrew Dunstan <andrew@dunslane.net>, pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2022-01-14T22:44:20Z
Lists: pgsql-hackers
Attachments
Hi, On 2022-01-15 10:59:00 +1300, Thomas Munro wrote: > On Sat, Jan 15, 2022 at 9:28 AM Andres Freund <andres@anarazel.de> wrote: > > I think it doesn't even need to touch socket.c to cause breakage. Using two > > different WaitEventSets is enough. > > Right. I was interested in your observation because so far we'd > *only* been considering the two-consecutive-WaitEventSets case, which > we could grok experimentally. There likely are further problems in other parts, but I think socket.c is unlikely to be involved in walreceiver case - there shouldn't be any socket.c style socket in walreceiver itself, nor do I think we are doing a send/recv/select backed by socket.c. > The patch Alexander tested most recently uses a tri-state eof flag [...] What about instead giving WalReceiverConn an internal WaitEventSet, and using that consistently? I've attached a draft for that. Alexander, could you test with that patch applied? Greetings, Andres Freund
Commits
-
Revert "graceful shutdown" changes for Windows.
- 29992a6a509b 15.0 landed
-
Revert "graceful shutdown" changes for Windows, in back branches only.
- bbb1caf6b220 11.15 landed
- 75674c7ec1b1 14.2 landed
- 64b2c6507e57 12.10 landed
- 645c9457f296 10.20 landed
- 213c5aa3bdba 13.6 landed