Re: optimize file transfer in pg_upgrade
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Nathan Bossart <nathandbossart@gmail.com>,
Greg Sabino Mullane <htamfids@gmail.com>,
pgsql-hackers@postgresql.org,
bruce@momjian.us
Date: 2025-03-18T14:04:41Z
Lists: pgsql-hackers
Robert Haas <robertmhaas@gmail.com> writes: > I'm not quite sure what the best thing is to do is for the pg_upgrade > tests in particular, and it may well be best to do as you propose for > now and figure that out later. But I question whether just rerunning > all of those tests with several different mode flags is the right > thing to do. Why for example does 005_char_signedness.pl need to be > checked under both --link and --clone? I would guess that there are > one or maybe two tests in src/bin/pg_upgrade/t that needs to test > --link and --clone and they should grow internal loops to do that > (when supported by the local platform) and PG_UPGRADE_TEST_MODE should > go in the garbage. +1 I'd be particularly allergic to running 002_pg_upgrade.pl multiple times, as that's one of our most expensive tests, and I flat out don't believe that expending that many cycles could be justified. Surely we can test these modes sufficiently in some much cheaper and more targeted way. regards, tom lane
Commits
-
initdb: Add --no-sync-data-files.
- cf131fa94285 18.0 landed
-
pg_dump: Add --sequence-data.
- 9c49f0e8cd7d 18.0 landed
-
pg_upgrade: Add --swap for faster file transfer.
- 626d7236b65d 18.0 landed
-
Add test for pg_upgrade file transfer modes.
- af0d4901c1c3 18.0 landed
-
Fix an intermetant BF failure in 003_logical_slots.
- 6f97ef05d62a 17.0 cited