pgsql: In basebackup.c, refactor to create read_file_data_into_buffer. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: In basebackup.c, refactor to create read_file_data_into_buffer.
Date
Msg-id E1qngti-007JiS-ND@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
In basebackup.c, refactor to create read_file_data_into_buffer.

This further reduces the length and complexity of sendFile(),
hopefully make it easier to understand and modify. In addition
to moving some logic into a new function, I took this opportunity
to make a few slight adjustments to sendFile() itself, including
renaming the 'len' variable to 'bytes_done', since we use it to represent
the number of bytes we've already handled so far, not the total
length of the file.

Patch by me, reviewed by David Steele.

Discussion: http://postgr.es/m/CA+TgmoYt5jXH4U6cu1dm9Oe2FTn1aae6hBNhZzJJjyjbE_zYig@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/c2ba3fdea5938dff96ca32ed7a4a83c8ff5f188d

Modified Files
--------------
src/backend/backup/basebackup.c | 231 +++++++++++++++++++++++-----------------
1 file changed, 133 insertions(+), 98 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: In basebackup.c, refactor to create verify_page_checksum.
Next
From: Tom Lane
Date:
Subject: pgsql: Add some notes about why "ALTER TYPE enum DROP VALUE" is hard.