pgsql: pg_basebackup: Refactor code for reading COPY and tar data. - Mailing list pgsql-committers

From Robert Haas
Subject pgsql: pg_basebackup: Refactor code for reading COPY and tar data.
Date
Msg-id E1icxW6-0003GQ-6K@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
pg_basebackup: Refactor code for reading COPY and tar data.

Add a new function ReceiveCopyData that does just that, taking a
callback as an argument to specify what should be done with each chunk
as it is received. This allows a single copy of the logic to be shared
between ReceiveTarFile and ReceiveAndUnpackTarFile, and eliminates
a few #ifdef conditions based on HAVE_LIBZ.

While this is slightly more code, it's arguably clearer, and
there is a pending patch that introduces additional calls to
ReceiveCopyData.

This commit is not intended to result in any functional change.

Discussion: http://postgr.es/m/CA+TgmoYZDTHbSpwZtW=JDgAhwVAYvmdSrRUjOd+AYdfNNXVBDg@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/431ba7bebf139b6edf5544ce18f39a1a4dcb8110

Modified Files
--------------
src/bin/pg_basebackup/pg_basebackup.c | 1005 +++++++++++++++++----------------
1 file changed, 507 insertions(+), 498 deletions(-)


pgsql-committers by date:

Previous
From: Robert Haas
Date:
Subject: pgsql: Minor comment improvements for instrumentation.h
Next
From: Michael Paquier
Date:
Subject: pgsql: Remove configure check for OpenSSL's SSL_get_current_compression