| From: | "Zhijie Hou (Fujitsu)" <houzj(dot)fnst(at)fujitsu(dot)com> |
|---|---|
| To: | 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:14:19 |
| Message-ID: | TY4PR01MB1690728DD69513D7C4D7317BF94B3A@TY4PR01MB16907.jpnprd01.prod.outlook.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
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)
Best Regards,
Hou zj
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Zhijie Hou (Fujitsu) | 2025-12-25 03:15:02 | RE: doc: update the default of data checksums in the doc of pg_rewind |
| Previous Message | Chao Li | 2025-12-25 03:12:07 | Trivial Fix: use palloc_array/repalloc_array for BufFile file arrays |