On 19 November 2014 13:13, Simon Riggs <simon@2ndquadrant.com> wrote:
> I've reworded docs a little.
Done
> If we ask for PAUSE and we're not in HotStandby it just ignores it,
> which means it changes into PROMOTE. My feeling is that we should
> change that into a SHUTDOWN, not a PROMOTE.
Done
>
> Also, for the Shutdown itself, why are we not using
> kill(PostmasterPid, SIGINT)?
Done
Other plan is to throw a FATAL message.
> That gives a clean, fast shutdown rather than what looks like a crash.
I've also changed the location of where we do
RECOVERY_TARGET_ACTION_SHUTDOWN, so its in the same place as where we
pause.
I've also moved the check to see if we should throw FATAL because
aren't yet consistent to *before* we do any actionOnRecoveryTarget
stuff. It seems essential that we know that earlier rather than later.
Thoughts?
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services