Make error handling in parallel pg_upgrade less bogus.
authorTom Lane <[email protected]>
Sun, 16 Dec 2018 19:51:47 +0000 (14:51 -0500)
committerTom Lane <[email protected]>
Sun, 16 Dec 2018 19:51:47 +0000 (14:51 -0500)
commit16fda4b853cd5bc07c450303a465ed9d451c7b47
tree777c8a984fa523c7482e6c6e18020a681eac49a2
parentade2d61ed09c11a3d374a98c66c8a05cb82a0069
Make error handling in parallel pg_upgrade less bogus.

reap_child() basically ignored the possibility of either an error in
waitpid() itself or a child process failure on signal.  We don't really
need to do more than report and crash hard, but proceeding as though
nothing is wrong is definitely Not Acceptable.  The error report for
nonzero child exit status was pretty off-point, as well.

Noted while fooling around with child-process failure detection
logic elsewhere.  It's been like this a long time, so back-patch to
all supported branches.
src/bin/pg_upgrade/parallel.c