From: | Thomas Munro <thomas(dot)munro(at)gmail(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Justin Pryzby <pryzby(at)telsasoft(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
Subject: | Re: pg_upgrade test failure |
Date: | 2023-01-05 03:11:00 |
Message-ID: | CA+hUKGJTM5FswRV-_JWzVLdw=Lq1pjNsvGfONRJp=dpR6YR81Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
On Wed, Dec 7, 2022 at 7:15 AM Andres Freund <andres(at)anarazel(dot)de> wrote:
> On 2022-11-08 01:16:09 +1300, Thomas Munro wrote:
> > So [1] on its own didn't fix this. My next guess is that the attached
> > might help.
> What is our plan here? This afaict is the most common "false positive" for
> cfbot in the last weeks.
That branch hasn't failed on cfbot[1], except once due to one of the
other known flapping races we have to fix. Which doesn't prove
anything, of course, but it is encouraging. I wish we knew why the
test does this, though....
Here's a better version that works harder to avoid opening more than
one fd at a time (like the pgfnames()-based code it replaces), and
also uses fd.c facilities in the backend version (unlike pgfnames(),
which looks like it could leak a descriptor if palloc() threw, and
also doesn't know how to handle file descriptor pressure).
[1] https://cirrus-ci.com/github/postgresql-cfbot/postgresql/commitfest/41/4011
Attachment | Content-Type | Size |
---|---|---|
v2-0001-Refactor-rmtree-to-use-get_dirent_type.patch | text/x-patch | 5.1 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2023-01-05 06:58:50 | Re: pgsql: Delay commit status checks until freezing executes. |
Previous Message | David Rowley | 2023-01-04 23:56:45 | pgsql: Fix some compiler warnings in aset.c and generation.c |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2023-01-05 03:12:04 | Re: Todo: Teach planner to evaluate multiple windows in the optimal order |
Previous Message | Tom Lane | 2023-01-05 02:48:05 | Re: Todo: Teach planner to evaluate multiple windows in the optimal order |