On Mon, Oct 23, 2017 at 10:27 PM, <t.katsumata1122@gmail.com> wrote: > At "recovery_target_inclusive (boolean)", nothing is mentioned about > "recovery_target_lsn". > It should be fixed, since "recovery_target_lsn" is affected by the value of > "recovery_target_inclusive". > > Could you check this?
Apologies. You are right, this is a documentation bug and should be fixed. Attached is a patch to address the problem. The trend on the documentation page for recovery parameters is to list the options alphabetically, so recovery_target_lsn comes first. -- Michael
Your patch seems good, but I have a question.
Does it need to fix the next sentence?
Like this:
----
This setting controls whether transactions having exactly the target WAL location (LSN), commit time or ID, respectively, will be included in the recovery.