]> granicus.if.org Git - postgresql/commit
Un-break parallel pg_upgrade.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 25 Feb 2018 22:27:20 +0000 (17:27 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 25 Feb 2018 22:27:20 +0000 (17:27 -0500)
commit5b570d771b80aadc98755208f8f1b81e9a5eb366
tree574de7d50469613ad259af9968edb85ceefeabce
parent1316417bbab0821f99eb21c0b654e33f5f6f90a4
Un-break parallel pg_upgrade.

Commit b3f840120 changed pg_upgrade so that it'd actually drop and
re-create the template1 and postgres databases in the new cluster.
That works fine, serially.  With the -j option it's not so fine, because
other per-database jobs might be launched while the template1 database is
dropped.  Since they attempt to connect there to start up, kaboom.

This is the cause of the intermittent failures buildfarm member jacana
has been showing for the last month; evidently it is the only BF member
configured to run the pg_upgrade test with parallelism enabled.

Fix by processing template1 separately before we get into the parallel
sub-job launch loop.  (We could alternatively have made the postgres DB
be the special case, but it seems likely that template1 will contain
less stuff and so we lose less parallelism with this choice.)
src/bin/pg_upgrade/pg_upgrade.c