Re: Timeline ID hexadecimal format

From: Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>
To: Sébastien Lardière <sebastien(at)lardiere(dot)net>, Greg Stark <stark(at)mit(dot)edu>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Timeline ID hexadecimal format
Date: 2023-03-02 08:12:41
Message-ID: b27d55d5-ab6f-8d62-04c4-4d00859fb7c9@enterprisedb.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 24.02.23 17:27, Sébastien Lardière wrote:
> diff --git a/doc/src/sgml/backup.sgml b/doc/src/sgml/backup.sgml
> index be05a33205..7e26b51031 100644
> --- a/doc/src/sgml/backup.sgml
> +++ b/doc/src/sgml/backup.sgml
> @@ -1332,7 +1332,8 @@ restore_command = 'cp/mnt/server/archivedir/%f %p'
> you like, add comments to a history file to record your own notes about
> how and why this particular timeline was created. Such comments will be
> especially valuable when you have a thicket of different timelines as
> - a result of experimentation.
> + a result of experimentation. In both WAL segment file names and history files,
> + the timeline ID number is expressed in hexadecimal.
> </para>
>
> <para>

I think here it would be more helpful to show actual examples. Like,
here is a possible file name, this is what the different parts mean.

> diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
> index e5c41cc6c6..3b5d041d92 100644
> --- a/doc/src/sgml/config.sgml
> +++ b/doc/src/sgml/config.sgml
> @@ -4110,7 +4110,9 @@ restore_command = 'copy "C:\\server\\archivedir\\%f" "%p"' # Windows
> current when the base backup was taken. The
> value <literal>latest</literal> recovers
> to the latest timeline found in the archive, which is useful in
> - a standby server. <literal>latest</literal> is the default.
> + a standby server. A numerical value expressed in hexadecimal must be
> + prefixed with <literal>0x</literal>, for example <literal>0x11</literal>.
> + <literal>latest</literal> is the default.
> </para>
>
> <para>

This applies to all configuration parameters, so it doesn't need to be
mentioned explicitly for individual ones.

> diff --git a/doc/src/sgml/ref/pg_waldump.sgml b/doc/src/sgml/ref/pg_waldump.sgml
> index 343f0482a9..4ae8f2ebdd 100644
> --- a/doc/src/sgml/ref/pg_waldump.sgml
> +++ b/doc/src/sgml/ref/pg_waldump.sgml
> @@ -215,7 +215,8 @@ PostgreSQL documentation
> <para>
> Timeline from which to read WAL records. The default is to use the
> value in <replaceable>startseg</replaceable>, if that is specified; otherwise, the
> - default is 1.
> + default is 1. The value must be expressed in decimal, contrary to the hexadecimal
> + value given in WAL segment file names and history files.
> </para>
> </listitem>
> </varlistentry>

Maybe this could be fixed instead?

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2023-03-02 08:17:34 Re: Show various offset arrays for heap WAL records
Previous Message shiy.fnst@fujitsu.com 2023-03-02 08:06:53 RE: [PATCH] Use indexes on the subscriber when REPLICA IDENTITY is full on the publisher