Make error handling in parallel pg_upgrade less bogus.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Dec 2018 19:51:48 +0000 (14:51 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Dec 2018 19:51:48 +0000 (14:51 -0500)
commitb10d2252b35326566d5520acbc65842927c4c3a1
tree30ebe7f04d0d6eb4c5d98fcb4c2474e55d810cf4
parentbf0e5a73be20b1f010b000c37c9980ac02809fb5
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.
contrib/pg_upgrade/parallel.c