Re: optimize file transfer in pg_upgrade
Nathan Bossart <nathandbossart@gmail.com>
From: Nathan Bossart <nathandbossart@gmail.com>
To: Robert Haas <robertmhaas@gmail.com>
Cc: Greg Sabino Mullane <htamfids@gmail.com>, pgsql-hackers@postgresql.org, bruce@momjian.us
Date: 2025-03-17T20:30:31Z
Lists: pgsql-hackers
Attachments
- v6-0001-initdb-Add-no-sync-data-files.patch (text/plain)
On Mon, Mar 17, 2025 at 04:04:45PM -0400, Robert Haas wrote: > On Mon, Mar 17, 2025 at 3:34 PM Nathan Bossart <nathandbossart@gmail.com> wrote: >> * Once committed, I should update one of my buildfarm animals to use >> PG_TEST_PG_UPGRADE_MODE=--swap. > > It would be better if we didn't need a separate buildfarm animal to > test this, because that means you won't get notified by local testing > OR by CI if you break this. Can we instead have one test that checks > this which is part of the normal test run? That's what I set out to do before I discovered PG_TEST_PG_UPGRADE_MODE. The commit message for b059a24 seemed to indicate that we don't want to automatically test all supported modes, but I agree that it would be nice to have some basic coverage for --swap on CI/buildfarm regardless of PG_TEST_PG_UPGRADE_MODE. How about we add a simple TAP test (attached), and I still plan on switching a buildfarm animal to --swap for more in-depth testing? -- nathan
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