Make error handling in parallel pg_upgrade less bogus.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Dec 2018 19:51:47 +0000 (14:51 -0500)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Dec 2018 19:51:47 +0000 (14:51 -0500)
commitf4290113f50d1bcd67a9e51b74927c59592ad2ea
tree8c10c6a0ea564243b6c0d6847d15615bcd2f6a63
parent34010ac2fa187ce032a7b243c829c7ef5f047e20
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