On Mon, Aug 27, 2018 at 07:34:55PM -0700, Michael Paquier wrote:
> That seems like a good catch and a correct fix to me. The handling of
> SIGINT is inconsistent with SIGTERM in pmdie(). I would just add a
> comment to mention that at this stage only the startup process is
> running, and that it has been signaled already. I'll commit that
> tomorrow.
I have been studying your patch, but it seems to me that this is not
complete as other processes could have been started before switching
from PM_STARTUP to PM_RECOVERY. I am talking here about the bgwriter
and the checkpointer as well. Shouldn't we switch pmState to
PM_WAIT_BACKENDS? Your patch is missing that.
--
Michael