|From:||Michael Paquier <michael(at)paquier(dot)xyz>|
|To:||James Coleman <jtc331(at)gmail(dot)com>|
|Cc:||PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>|
|Subject:||Re: pg_rewind docs correction|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
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".
> 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
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.
|Next Message||Rui DeSousa||2020-04-29 00:20:53||Re: PostgreSQL CHARACTER VARYING vs CHARACTER VARYING (Length)|
|Previous Message||raf||2020-04-28 23:49:56||Re: PostgreSQL CHARACTER VARYING vs CHARACTER VARYING (Length)|