On Tue, Apr 28, 2020 at 12:13:38PM -0400, James Coleman wrote:
> I think is missing a word. Instead of "especially the the target"
> should be "especially if the target".
Thanks, fixed.
> In this block:
>
> + Create a <filename>backup_label</filename> file to begin WAL replay at
> + the checkpoint created at failover and configure the
> + <filename>pg_control</filename> file with a minimum consistency LSN
> + defined as the result of <literal>pg_current_wal_insert_lsn()</literal>
> + when rewinding from a live source and using the last checkpoint LSN
> + when rewinding from a stopped source.
> + </para>
>
> Perhaps change "and using the last checkpoint LSN" to "or the last
> checkpoint LSN". Alternatively you could make the grammar parallel by
> changing to "and defined as the last checkpoint LSN", but that seems
> wordy, and the "defined as [item or item]" is already a good grammar
> construction.
Using your first suggestion of "or the last checkpoint LSN" sounds
more natural as of this morning, so updated the patch with that.
I am letting that aside for a couple of days to see if others have
more comments, and will likely commit it after an extra lookup.
--
Michael
