pgdump/parallel.c: "aborting" flag is dead code

Started by Tom Lanealmost 10 years ago1 messageshackers
Jump to latest
#1Tom Lane
tgl@sss.pgh.pa.us

parallel.c has an "aborting" flag that's entirely useless, as it's
set only during archive_close_connection(), and tested only in places
that are not reached after that. I've confirmed this both by code
reading and by testing. It appears from some of the comments that
there was once an intent to shut down workers by sending an explicit
TERMINATE message, rather than just closing the pipe, and in that
case maybe there would have been some value in this flag. As is,
though, it just sows confusion, so I propose removing it as attached.

regards, tom lane

Attachments:

remove-useless-aborting-flag.patchtext/x-diff; charset=us-ascii; name=remove-useless-aborting-flag.patchDownload+26-34