Re: pgsql: Prefetch data referenced by the WAL, take II. - Mailing list pgsql-committers

From Andrew Dunstan
Subject Re: pgsql: Prefetch data referenced by the WAL, take II.
Date
Msg-id ef12f73f-3d85-75bf-6728-6f03b77c8d61@dunslane.net
Whole thread Raw
In response to Re: pgsql: Prefetch data referenced by the WAL, take II.  (Andres Freund <andres@anarazel.de>)
Responses Re: pgsql: Prefetch data referenced by the WAL, take II.  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-committers
On 4/7/22 04:36, Andres Freund wrote:
> Hi,
>
> On 2022-04-07 07:44:20 +0000, Thomas Munro wrote:
>> Prefetch data referenced by the WAL, take II.
> https://buildfarm.postgresql.org/cgi-bin/show_log.pl?nm=crake&dt=2022-04-07%2008%3A17%3A27
> thinks that xlogpretcher.h doesn't include enough...
>
> I had added those checks to CI, but apparently somehow screwed that up.
>
> Or maybe it's the scripts that are screwed up? Because I don't see any error
> checking in headerscheck/cpluspluscheck. And indeed, locally they show the
> errors, but exit with 0.


Yeah, you can't rely on the exit status, if they produce output that
should be regarded as a failure (that's what crake does).


cheers


andrew


--
Andrew Dunstan
EDB: https://www.enterprisedb.com




pgsql-committers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: pgsql: Fix typo in xlogrecovery.c code comment
Next
From: Tom Lane
Date:
Subject: Re: pgsql: Prefetch data referenced by the WAL, take II.