Re: doc review for v14

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Justin Pryzby <pryzby(at)telsasoft(dot)com>
Cc: pgsql-hackers(at)postgresql(dot)org, Bruce Momjian <bruce(at)momjian(dot)us>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Subject: Re: doc review for v14
Date: 2021-03-01 06:17:40
Message-ID: YDyHBBYSme6WvmYh@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Feb 28, 2021 at 10:33:55PM -0600, Justin Pryzby wrote:
> I still have an issue with the sentence that begins:
> "Checksums are normally enabled..."

You could say here "Checksums can be enabled", but "normally" does not
sound bad to me either as it insists on the fact that it is better to
do that when the cluster is initdb'd as this has no downtime compared
to enabling checksums on an existing cluster.

> Note, the patch I sent said "create" but should be "created".

Initialized sounds better to me, FWIW.

> "page corruptions" is wrong .. you could say "corrupt pages"

"corruptED pages" would sound more correct to me as something that has
already happened. Anyway, I'd rather keep what I am proposing
upthread.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2021-03-01 06:30:44 Re: [PATCH] refactor ATExec{En,Dis}ableRowSecurity
Previous Message Corey Huinker 2021-03-01 06:13:47 Re: simplifying foreign key/RI checks