Re: optimizing pg_upgrade's once-in-each-database steps
Nathan Bossart <nathandbossart@gmail.com>
On Thu, Aug 08, 2024 at 06:18:38PM -0400, Corey Huinker wrote: > I think the underlying mechanism is basically solid, but I have one > question: isn't this the ideal case for using libpq pipelining? That would > allow subsequent tasks to launch while the main loop slowly gets around to > clearing off completed tasks on some other connection. I'll admit I hadn't really considered pipelining, but I'm tempted to say that it's probably not worth the complexity. Not only do most of the tasks have only one step, but even tasks like the data types check are unlikely to require more than a few queries for upgrades from supported versions. Furthermore, most of the callbacks should do almost nothing for a given upgrade, and since pg_upgrade runs on the server, client/server round-trip time should be pretty low. Perhaps pipelining would make more sense if we consolidated the tasks a bit better, but when I last looked into that, I didn't see a ton of great opportunities that would help anything except for upgrades from really old versions. Even then, I'm not sure if pipelining is worth it. -- nathan
Commits
-
Introduce framework for parallelizing various pg_upgrade tasks.
- 40e2e5e92b7d 18.0 landed
-
pg_upgrade: Parallelize WITH OIDS check.
- f93f5f7b984e 18.0 landed
-
pg_upgrade: Parallelize contrib/isn check.
- 9db3018cf83b 18.0 landed
-
pg_upgrade: Parallelize data type checks.
- bbf83cab98f2 18.0 landed
-
pg_upgrade: Parallelize encoding conversion check.
- c880cf258864 18.0 landed
-
pg_upgrade: Parallelize incompatible polymorphics check.
- cf2f82a37cc3 18.0 landed
-
pg_upgrade: Parallelize postfix operator check.
- c34eabfbbfd3 18.0 landed
-
pg_upgrade: Parallelize retrieving extension updates.
- 6ab8f27bc716 18.0 landed
-
pg_upgrade: Parallelize retrieving loadable libraries.
- 46cad8b31927 18.0 landed
-
pg_upgrade: Parallelize retrieving relation information.
- 6d3d2e8e541f 18.0 landed
-
pg_upgrade: Parallelize subscription check.
- 7baa36de58bd 18.0 landed
-
pg_upgrade: Move live_check variable to user_opts.
- 4b56bb4ab485 18.0 landed
-
pg_upgrade: run all data type checks per connection
- 347758b12063 17.0 cited