pgsql: Apply filters to dump files all the time in 002_pg_upgrade.pl - Mailing list pgsql-committers

From Michael Paquier
Subject pgsql: Apply filters to dump files all the time in 002_pg_upgrade.pl
Date
Msg-id E1rAgIX-008nha-E0@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Apply filters to dump files all the time in 002_pg_upgrade.pl

This commit removes the restriction that would not apply filters to the
dumps used for comparison in the TAP test of pg_upgrade when using the
same base version for the old and new nodes.

The previous logic would fail on Windows if loading a dump while using
the same set of binaries for the old and new nodes, as the library
dependencies updated in the old dump would append CRLFs to the dump
file as it is treated as a text file.  The dump filtering logic replaces
all CRLFs (\r\n) by LFs (\n), which is able to prevent this issue.

When the old and new versions of the binaries are the same,
AdjustUpgrade removes all blank lines, removes version-based comments
generated by pg_dump and replaces CRLFs by LFs.

Reported-by: Alexander Lakhin
Discussion: https://postgr.es/m/60d434b9-53d9-9ea1-819b-efebdcf44e41@gmail.com
Backpatch-through: 15

Branch
------
REL_16_STABLE

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

Modified Files
--------------
src/bin/pg_upgrade/t/002_pg_upgrade.pl | 12 +++---------
1 file changed, 3 insertions(+), 9 deletions(-)


pgsql-committers by date:

Previous
From: Alexander Korotkov
Date:
Subject: pgsql: Add support for deparsing semi-joins to contrib/postgres_fdw
Next
From: Michael Paquier
Date:
Subject: pgsql: Fix compilation on Windows with WAL_DEBUG