pgsql: Make error handling in parallel pg_upgrade less bogus. - Mailing list pgsql-committers

From Tom Lane
Subject pgsql: Make error handling in parallel pg_upgrade less bogus.
Date
Msg-id E1gYcSM-0008P1-OF@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
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.

Branch
------
REL9_6_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/f7642cf31601b206cb7cfb99339e6b4edfec0b58

Modified Files
--------------
src/bin/pg_upgrade/parallel.c | 17 ++++++++---------
1 file changed, 8 insertions(+), 9 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Make pg_statistic and related code account more honestly forcol
Next
From: Tom Lane
Date:
Subject: pgsql: Modernize our code for looking up descriptive strings for Unixs