Re: reaper should restart archiver even on standby

Fujii Masao wrote:

> Agreed. The attached patch defines the macro to check whether archiver is
> allowed to start up or not, and uses it everywhere except sigusr1_handler.
> I made sigusr1_handler use a different condition because only it tries to
> start archiver in PM_STARTUP postmaster state and it looks a bit messy
> to add the check of that state into the centralized check condition.

WFM, but do these macros in xlog.h need a one-line comment to state
their purpose?

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services