Thomas Munro <thomas.munro@gmail.com> writes:
> Here's an experimental patch to fix our shutdown strategy on
> postmaster death, as discussed in a nearby report[1].
Thanks for tackling this topic.
> For systems lacking that facility, the idea I'm trying out here is
> that backends that detect the condition in WaitEventSetWait() should
> themselves blast all backends with SIGQUIT, in a sense taking over the
> role of the departed postmaster.
Hmm. Up to now, we have not had an assumption that postmaster
children are aware of every other postmaster child. In particular,
not all postmaster children have PGPROC entries. How much does
this matter? What happens if the shared PGPROC array is corrupt?
> I didn't really want any
> consensus/negotiation over who's going to do that, so... they all do.
Agreed on that point.
> Most of the patch is just removing hundreds of lines of errors and
> conditions and comments that were now unreachable.
The patch would likely be a lot more readable if you split out the
"delete unreachable code" part into a separate step.
regards, tom lane
