From: Daniel Gustafsson <daniel@yesql.se>
Content-Type: multipart/mixed;
	boundary="Apple-Mail=_8E867BCA-0FC6-4968-B8D7-FBF9D6D86640"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3776.700.51.11.12\))
Subject: Re: 66.6. Database Page Layout
Date: Thu, 9 Jul 2026 23:58:55 +0200
References: <178350739237.73862.4549076173872335741@wrigleys.postgresql.org>
To: y.saburov@gmail.com,
 PostgreSQL-documentation <pgsql-docs@lists.postgresql.org>
In-Reply-To: <178350739237.73862.4549076173872335741@wrigleys.postgresql.org>
Message-Id: <24B34848-3557-48E0-B601-A5BA4FC4E057@yesql.se>
Archived-At: 
 <https://www.postgresql.org/message-id/24B34848-3557-48E0-B601-A5BA4FC4E057%40yesql.se>
Precedence: bulk


--Apple-Mail=_8E867BCA-0FC6-4968-B8D7-FBF9D6D86640
Content-Transfer-Encoding: quoted-printable
Content-Type: text/plain;
	charset=us-ascii

> On 8 Jul 2026, at 12:43, PG Doc comments form <noreply@postgresql.org> =
wrote:
>=20
> The following documentation comment has been logged on the website:
>=20
> Page: https://www.postgresql.org/docs/18/storage-page-layout.html
> Description:
>=20
>> The second field contains the page checksum if -k are enabled.
>=20
> -k -- This is enabled by default; use --no-data-checksums to disable
> checksums.
>=20
> Why does this need to be clarified?

I'm not entirely sure which part you are referring to, but I agree that =
the
current wording isn't very good.

	"The second field contains the page checksum if -k are enabled."

I propose the attached patch which changes the sentence to "The second =
field
contains the page checksum if data checksums are enabled." with a link =
to the
Checksums chapter in the documentation instead.

--
Daniel Gustafsson


--Apple-Mail=_8E867BCA-0FC6-4968-B8D7-FBF9D6D86640
Content-Disposition: attachment;
	filename=page_layout_checksums.diff
Content-Type: application/octet-stream;
	x-unix-mode=0644;
	name="page_layout_checksums.diff"
Content-Transfer-Encoding: 7bit

diff --git a/doc/src/sgml/storage.sgml b/doc/src/sgml/storage.sgml
index 6b6377503bf..19924b98d71 100644
--- a/doc/src/sgml/storage.sgml
+++ b/doc/src/sgml/storage.sgml
@@ -797,7 +797,7 @@ data. Empty in ordinary tables.</entry>
   (<structname>PageHeaderData</structname>). Its format is detailed in <xref
   linkend="pageheaderdata-table"/>. The first field tracks the most
   recent WAL entry related to this page. The second field contains
-  the page checksum if <xref linkend="app-initdb-data-checksums"/> are
+  the page checksum if <link linkend="checksums">data checksums</link> are
   enabled.  Next is a 2-byte field containing flag bits. This is followed
   by three 2-byte integer fields (<structfield>pd_lower</structfield>,
   <structfield>pd_upper</structfield>, and

--Apple-Mail=_8E867BCA-0FC6-4968-B8D7-FBF9D6D86640--