On Thu, Sep 28, 2023 at 12:58:51PM +0900, Kyotaro Horiguchi wrote: > The attached is a quick mock-up, but providing an approximation of my > thoughts. (For example, end_of_backup_reached could potentially be > extended to the ArchiveRecoveryRequested case and we could simplify > the condition..)
I am not sure why this is related to this thread.. static XLogRecPtr backupStartPoint; static XLogRecPtr backupEndPoint; static bool backupEndRequired = false; +static bool backupEndReached = false; Anyway, sneaking at your suggestion, this is actually outlining the main issue I have with this code currently. We have so many static booleans to control one behavior over the other that we always try to make this code more complicated, while we should try to make it simpler instead. -- Michael
signature.asc
Description: PGP signature