WAL Configuration Grammar Fix

Started by PG Bug reporting formabout 7 years ago2 messagesdocs
Jump to latest
#1PG Bug reporting form
noreply@postgresql.org

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/11/wal-configuration.html
Description:

On the WAL Configuration page -
https://www.postgresql.org/docs/current/wal-configuration.html
It reads: "On Linux and POSIX platforms checkpoint_flush_after allows to
force the OS that pages written by the checkpoint should be flushed to disk
after a configurable number of bytes."

I believe a "you" is missing. I also suggest a more clear sentence
construction might be:
"On Linux and POSIX platforms you can use checkpoint_flush_after to set a
configurable number of bytes after which the OS will be forced to flush to
disk the pages written by the checkpoint."

Respectfully submitted.
Evan Macbeth

#2Bruce Momjian
bruce@momjian.us
In reply to: PG Bug reporting form (#1)
Re: WAL Configuration Grammar Fix

On Fri, Mar 8, 2019 at 10:39:16PM +0000, PG Doc comments form wrote:

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/11/wal-configuration.html
Description:

On the WAL Configuration page -
https://www.postgresql.org/docs/current/wal-configuration.html
It reads: "On Linux and POSIX platforms checkpoint_flush_after allows to
force the OS that pages written by the checkpoint should be flushed to disk
after a configurable number of bytes."

I believe a "you" is missing. I also suggest a more clear sentence
construction might be:
"On Linux and POSIX platforms you can use checkpoint_flush_after to set a
configurable number of bytes after which the OS will be forced to flush to
disk the pages written by the checkpoint."

This patch is 4.5 years old, but still valid. Fixed with the attached
patch applied to master.

--
Bruce Momjian <bruce@momjian.us> https://momjian.us
EDB https://enterprisedb.com

Only you can decide what is important to you.

Attachments:

master.difftext/x-diff; charset=us-asciiDownload+1-1