RE: doc: update the default of data checksums in the doc of pg_rewind

From: "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com>
To: "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: RE: doc: update the default of data checksums in the doc of pg_rewind
Date: 2025-12-25 03:16:34
Message-ID: TY4PR01MB16907823AC9D4237641FE28EA94B3A@TY4PR01MB16907.jpnprd01.prod.outlook.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thursday, December 25, 2025 11:14 AM Zhijie Hou (Fujitsu) <houzj(dot)fnst(at)fujitsu(dot)com> wrote:
> On Thursday, December 25, 2025 8:22 AM Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>
> wrote:
> >
> > > On Dec 24, 2025, at 15:27, Zhijie Hou (Fujitsu)
> > > <houzj(dot)fnst(at)fujitsu(dot)com>
> > wrote:
> > >
> > > Hi,
> > >
> > > I noticed that the pg_rewind documentation still states that data
> > > checksums is disabled by default, but it has been changed to on since
> PG18.
> > >
> > > Here's a small patch to update this.
> > >
> > > Best Regards,
> > > Hou zj
> > > <v1-0001-Fix-outdated-doc-in-pg_rewind.patch>
> >
> >
> > ```
> > - the cluster was initialized with <application>initdb</application>.
> Neither
> > of these
> > - are currently on by default. <xref linkend="guc-full-page-writes"/>
> > - must also be set to <literal>on</literal>, but is enabled by default.
> > + the cluster was initialized with <application>initdb</application>
> > + (the
> > latter
> > + being the default). <xref linkend="guc-full-page-writes"/> must also be
> set
> > + to <literal>on</literal>, but is enabled by default.
> > ```
> >
> > I think we can explicitly mention PG18 and later, like:
> > ```
> > the cluster was initialized with <application>initdb</application>
> > (data checksums are enabled by default in PostgreSQL 18 and later).
> > ```
> >
> > I searched over the docs, the usage of “<version> and later” is not
> > rare, for
> > example:
> > ```
> > <para>
> > <application>pg_amcheck</application> is designed to work with
> > <productname>PostgreSQL</productname> 14.0 and later.
> > </para>
> > ```
>
> I think including a version number here is necessary, as pg_rewind is not
> compatible with older PostgreSQL versions. It should be the release notes of
> PostgreSQL 18 to reflect changes in default values, rather than the pg_rewind
> doc mentioning a behavior change. (We don't mention version number in
> initdb's doc as well)

Sorry, a typo above, 'necessary' -> 'unnecessary'.

Best Regards,
Hou zj

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Chao Li 2025-12-25 03:34:48 Re: Trivial Fix: use palloc_array/repalloc_array for BufFile file arrays
Previous Message Zhijie Hou (Fujitsu) 2025-12-25 03:15:02 RE: doc: update the default of data checksums in the doc of pg_rewind