Behavior of recovery_target_inclusive.

Started by Tomonari Katsumataalmost 9 years ago6 messagesdocs
Beta feature

Hackorum builds and tests every patch posted to the lists, not only commitfest submissions. This is Hackorum's own CI rather than the PostgreSQL project's, and it is still under testing - please report anything that looks wrong.

won't retrysuccessCI history

You can run a PostgreSQL built from this patch straight from Docker, with no checkout and no build:

docker run --rm -p 5432:5432 ghcr.io/hackorum-dev/postgres-patch:t76063
psql -h localhost -U postgres

Built from patchset v5 (message #5), July 27, 2026 at 09:59 PM.

Every patchset is also pushed to a branch of our PostgreSQL fork, so you can check out the same tree CI built. Without a PostgreSQL checkout:

git clone --branch t76063_5 https://github.com/hackorum-dev/postgres.git

In a checkout you already have, add the fork once:

git remote add hackorum https://github.com/hackorum-dev/postgres.git

then, for this patchset and every later one:

git fetch hackorum t76063_5 && git checkout t76063_5

Patchset v5 (message #5) is on t76063_5

Jump to latest
#1Tomonari Katsumata
t.katsumata1122@gmail.com

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/10/static/index.html
Description:

Hi,

I found a small bug in the PostgreSQL 10 document.
https://www.postgresql.org/docs/10/static/recovery-target-settings.html

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?

Best regards,

--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs

#2Michael Paquier
michael@paquier.xyz
In reply to: Tomonari Katsumata (#1)
Re: Behavior of recovery_target_inclusive.

On Mon, Oct 23, 2017 at 10:27 PM, <t.katsumata1122@gmail.com> wrote:

At &quot;recovery_target_inclusive (boolean)&quot;, nothing is mentioned about
&quot;recovery_target_lsn&quot;.
It should be fixed, since &quot;recovery_target_lsn&quot; is affected by the value of
&quot;recovery_target_inclusive&quot;.

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

Attachments:

recovery-lsn-doc.patchapplication/octet-stream; name=recovery-lsn-doc.patchDownload+3-2
#3Tomonari Katsumata
t.katsumata1122@gmail.com
In reply to: Michael Paquier (#2)
Re: Behavior of recovery_target_inclusive.

Hi Michael

Thanks for your reply and quick reaction.

2017-10-23 23:05 GMT+09:00 Michael Paquier <michael.paquier@gmail.com>:

On Mon, Oct 23, 2017 at 10:27 PM, <t.katsumata1122@gmail.com> wrote:

At &quot;recovery_target_inclusive (boolean)&quot;, nothing is

mentioned about

&quot;recovery_target_lsn&quot;.
It should be fixed, since &quot;recovery_target_lsn&quot; is affected

by the value of

&quot;recovery_target_inclusive&quot;.

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.
----

Best regards,

#4Michael Paquier
michael@paquier.xyz
In reply to: Tomonari Katsumata (#3)
Re: Behavior of recovery_target_inclusive.

On Tue, Oct 24, 2017 at 9:43 AM, Tomonari Katsumata
<t.katsumata1122@gmail.com> wrote:

Does it need to fix the next sentence?

Right.
--
Michael

--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs

#5Tomonari Katsumata
t.katsumata1122@gmail.com
In reply to: Michael Paquier (#4)
Re: Behavior of recovery_target_inclusive.

Hi,

I rivised the patch.
Please update the file with this patch.

Best regards,

2017-10-24 9:45 GMT+09:00 Michael Paquier <michael.paquier@gmail.com>:

Show quoted text

On Tue, Oct 24, 2017 at 9:43 AM, Tomonari Katsumata
<t.katsumata1122@gmail.com> wrote:

Does it need to fix the next sentence?

Right.
--
Michael

Attachments:

t76063_5
recovery-lsn-doc2.patchapplication/octet-stream; name=recovery-lsn-doc2.patchDownload+4-3
#6Peter Eisentraut
peter_e@gmx.net
In reply to: Tomonari Katsumata (#5)
Re: Behavior of recovery_target_inclusive.

On 10/23/17 23:59, Tomonari Katsumata wrote:

I rivised the patch.
Please update the file with this patch.

committed to master and PG 10

--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

--
Sent via pgsql-docs mailing list (pgsql-docs@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-docs