Re: WAL Configuration Grammar Fix

2023-11-09 Thread Bruce Momjian
On Fri, Mar  8, 2019 at 10:39:16PM +, 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  https://momjian.us
  EDB  https://enterprisedb.com

  Only you can decide what is important to you.
diff --git a/doc/src/sgml/wal.sgml b/doc/src/sgml/wal.sgml
index 4aad0e1a07..184404c6c5 100644
--- a/doc/src/sgml/wal.sgml
+++ b/doc/src/sgml/wal.sgml
@@ -602,7 +602,7 @@
 
   
On Linux and POSIX platforms 
-   allows to force the OS that pages written by the checkpoint should be
+   allows you to force OS pages written by the checkpoint to be
flushed to disk after a configurable number of bytes.  Otherwise, these
pages may be kept in the OS's page cache, inducing a stall when
fsync is issued at the end of a checkpoint.  This setting will


WAL Configuration Grammar Fix

2019-03-08 Thread PG Doc comments form
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