Re: pgsql: Prefetch data referenced by the WAL, take II.

Andrew Dunstan <andrew@dunslane.net>

From: Andrew Dunstan <andrew@dunslane.net>
To: Andres Freund <andres@anarazel.de>, Thomas Munro <tmunro@postgresql.org>
Cc: pgsql-committers@lists.postgresql.org
Date: 2022-04-07T12:18:05Z
Lists: pgsql-hackers
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




Commits

  1. Improve wal_decode_buffer_size description some more

  2. Include some missing headers.

  3. Prefetch data referenced by the WAL, take II.