out-of-date comment in CreateRestartPoint() - Mailing list pgsql-hackers

From Robert Haas
Subject out-of-date comment in CreateRestartPoint()
Date
Msg-id AANLkTimsgj3VuAWIj_aQnZr17ku07dBqURtEDTHxp9aK@mail.gmail.com
Whole thread Raw
Responses Re: out-of-date comment in CreateRestartPoint()  (Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>)
List pgsql-hackers
This comment obviously requires adjustment now that HS is committed.
The obvious way to change it is to replace "when we get hot standby
capability" with "when running in Hot Standby mode", but I'm not clear
whether that's all that's required.
       /*        * If the last checkpoint record we've replayed is already our last        * restartpoint, we can't
performa new restart point. We still update        * minRecoveryPoint in that case, so that if this is a shutdown
restart       * point, we won't start up earlier than before. That's not strictly        * necessary, but when we get
hotstandby capability, it would be rather        * weird if the database opened up for read-only connections at a
* point-in-time before the last shutdown. Such time travel is still        * possible in case of immediate shutdown,
though.       *        * We don't explicitly advance minRecoveryPoint when we do create a        * restartpoint. It's
assumedthat flushing the buffers will
 
do that as a        * side-effect.        */

Thoughts?

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company


pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Stefan's bug (was: max_standby_delay considered harmful)
Next
From: Simon Riggs
Date:
Subject: Re: Synchronization levels in SR