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 -- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
