On Fri, Jul 19, 2019 at 12:59:43AM -0400, Tom Lane wrote: > Hm, I think 0ba06e0 is actually the relevant change here? Though > 40cfe86 was a necessary cleanup fix.
Oops. Yes, I meant that. > I'm too tired to dig in the buildfarm database to be sure, but my > impression is that the failure rate is much-better-but-not-zero. > So I'd support back-patching those two commits, but I'm not sure > if that's the end of the conversation. Yeah, I am a bit afraid to miss something under the hood... Let's see... Looking at the last 90 days of failures, I actually see one from 63 days ago from bowerbird which seems to indicate that we are not yet completely done with it: https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=bowerbird&dt=2019-05-16%2010%3A30%3A52 # Running: pg_ctl stop -D H:\prog\bf\root\HEAD\pgsql.build\src\bin\pg_ctl\tmp_check\tmp_test_qB5F/data waiting for server to shut down....pg_ctl: could not open PID file "H:/prog/bf/root/HEAD/pgsql.build/src/bin/pg_ctl/tmp_check/tmp_test_qB5F/data/postmaster.pid": Permission denied not ok 16 - pg_ctl stop And there is this one from jacana with subscription tests from 27 days ago: https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=jacana&dt=2019-06-12%2003%3A42%3A55 This makes it at least two, still that's much less than before :( -- Michael
signature.asc
Description: PGP signature