Re: macOS prefetching support
Peter Eisentraut <peter@eisentraut.org>
From: Peter Eisentraut <peter@eisentraut.org>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2024-08-28T06:21:04Z
Lists: pgsql-hackers
On 26.08.24 07:54, Thomas Munro wrote: > On Sat, Aug 24, 2024 at 12:28 AM Peter Eisentraut <peter@eisentraut.org> wrote: >> In terms of $subject, this patch seems sufficient for now. > > WFM. I noticed you don't have an EINTR retry loop, but the man page > doesn't say you need one, so overall this patch LGTM. > > + * posix_fadvise() is the simplest standardized interface that accomplishes > + * this. We could add an implementation using libaio in the future; but note > + * that this API is inappropriate for libaio, which wants to have a buffer > + * provided to read into. > > I would consider just dropping that comment about libaio, if touching > this paragraph. Proposals exist for AIO of course, but not with > libaio, and better predictions with useful discussion of the topic > seem unlikely to fit in this margin. committed with that change
Commits
-
Standardize "read-ahead advice" terminology.
- 813fde73d4dd 18.0 landed
-
Fixup for prefetching support on macOS
- d7fe02fb9e6e 18.0 landed
-
Add prefetching support on macOS
- 6654bb92047b 18.0 landed