Re: improving basebackup.c's file-reading code - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: improving basebackup.c's file-reading code
Date
Msg-id 96F2A9C6-F184-448F-AEED-912860BBE15D@yesql.se
Whole thread Raw
In response to Re: improving basebackup.c's file-reading code  (Robert Haas <robertmhaas@gmail.com>)
Responses Re: improving basebackup.c's file-reading code  (Robert Haas <robertmhaas@gmail.com>)
List pgsql-hackers
> On 17 Jun 2020, at 17:45, Robert Haas <robertmhaas@gmail.com> wrote:
>
> On Wed, Apr 29, 2020 at 5:52 AM Hamid Akhtar <hamid.akhtar@gmail.com> wrote:
>> The idea and the patch looks good to me.
>>
>> It makes sense to change fread to basebackup_read_file which internally calls pg_pread which is a portable function
asopposed to read. As you've mentioned, this is much better for error handling. 
>
> Thanks for the review. I have now committed the patch, after rebasing
> and adjusting one comment slightly.

As this went in, can we close the 2020-07 CF entry or is there anything left in
the patchseries?

cheers ./daniel


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: More tzdb fun: POSIXRULES is being deprecated upstream
Next
From: Robert Haas
Date:
Subject: Re: Why forbid "INSERT INTO t () VALUES ();"