Re: pg_preadv() and pg_pwritev() - Mailing list pgsql-hackers

From Thomas Munro
Subject Re: pg_preadv() and pg_pwritev()
Date
Msg-id CA+hUKGL7yLpG45aOc4go1HQfhnr0pHWJMA-Vu2CxHG1Y5Z0F0g@mail.gmail.com
Whole thread Raw
In response to Re: pg_preadv() and pg_pwritev()  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Thu, Jan 14, 2021 at 9:26 AM Tom Lane <tgl@sss.pgh.pa.us> wrote:
> * You need to remove pread.o and pwrite.o from the hard-wired
> part of the list in src/port/Makefile, else they get built
> whether needed or not.

Right, done.

> * I don't much like this in fd.h:
>
> @@ -46,6 +46,7 @@
>  #include <dirent.h>
>
>
> +struct iovec;
>  typedef int File;
>
> because it makes it look like iovec and File are of similar
> status, which they hardly are.  Perhaps more like
>
>  #include <dirent.h>
> +
> +struct iovec;                  /* avoid including sys/uio.h here */

Done, except I wrote port/pg_iovec.h.

> I confirm clean builds on Big Sur and Catalina with this.

Thanks for checking.  I also checked on Windows via CI.  Pushed.



pgsql-hackers by date:

Previous
From: Stephen Frost
Date:
Subject: Re: Change default of checkpoint_completion_target
Next
From: James Coleman
Date:
Subject: Re: [DOC] Document concurrent index builds waiting on each other