Обсуждение: comment needs to be updated for HS?
postmaster.c contains the following comment just above the definition of PMState. It appears to be out of date: * After reaching a consistent point in WAL redo, startup process signals* us again, and we switch to PM_RECOVERY_CONSISTENTstate. There's currently* no difference between PM_RECOVERY and PM_RECOVERY_CONSISTENT, but we* couldstart accepting connections to perform read-only queries at this* point, if we had the infrastructure to do that. The next paragraph has what I believe to be the correct information: * Normal child backends can only be launched when we are in PM_RUN or* PM_RECOVERY_CONSISTENT state. (We also allow launchof normal I am happy to fix this but thought SImon or Heikki or Tom might like to either (a) jump in and do it themselves or (b) tell me if I've misunderstood the situation. If neither of those is the case then I will go ahead and fix it. Thanks, -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company
On Thu, May 13, 2010 at 4:20 AM, Robert Haas <robertmhaas@gmail.com> wrote: > postmaster.c contains the following comment just above the definition > of PMState. It appears to be out of date: > > * After reaching a consistent point in WAL redo, startup process signals > * us again, and we switch to PM_RECOVERY_CONSISTENT state. There's currently > * no difference between PM_RECOVERY and PM_RECOVERY_CONSISTENT, but we > * could start accepting connections to perform read-only queries at this > * point, if we had the infrastructure to do that. But the first sentence of the above seems to be correct and helpful. No? Regards, -- Fujii Masao NIPPON TELEGRAPH AND TELEPHONE CORPORATION NTT Open Source Software Center
On Wed, May 12, 2010 at 9:59 PM, Fujii Masao <masao.fujii@gmail.com> wrote: > On Thu, May 13, 2010 at 4:20 AM, Robert Haas <robertmhaas@gmail.com> wrote: >> postmaster.c contains the following comment just above the definition >> of PMState. It appears to be out of date: >> >> * After reaching a consistent point in WAL redo, startup process signals >> * us again, and we switch to PM_RECOVERY_CONSISTENT state. There's currently >> * no difference between PM_RECOVERY and PM_RECOVERY_CONSISTENT, but we >> * could start accepting connections to perform read-only queries at this >> * point, if we had the infrastructure to do that. > > But the first sentence of the above seems to be correct and helpful. No? Yes. I was just quoting the whole thing for context. -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise Postgres Company