Subject: Re: Fix typos and inconsistencies for HEAD (take 7)
To: Tom Lane <tgl@sss.pgh.pa.us>, Michael Paquier <michael@paquier.xyz>
Cc: pgsql-hackers@postgresql.org
References: <dff75442-2468-f74f-568c-6006e141062f@gmail.com>
 <20190722010543.GB1757@paquier.xyz> <28139.1563768879@sss.pgh.pa.us>
From: Alexander Lakhin <exclusion@gmail.com>
Message-ID: <fa7aea00-f26f-dad7-8d12-63fee14c36ed@gmail.com>
Date: Mon, 22 Jul 2019 08:00:00 +0300
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.7.2
MIME-Version: 1.0
In-Reply-To: <28139.1563768879@sss.pgh.pa.us>
Content-Type: multipart/alternative;
 boundary="------------DEAE41C3D7AA538074018D8C"
Content-Language: en-US
Precedence: bulk

This is a multi-part message in MIME format.
--------------DEAE41C3D7AA538074018D8C
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit

Hello Tom,
22.07.2019 7:14, Tom Lane wrote:
>> Fixing both places sounds adapted to me.  An alternative we could use
>> here is just to say something like that:
>> The effective resolution is only 1/HZ, which can be configured with
>> kernel parameter (see man 7 time), and is 4 milliseconds by
>> default.
> Whatever we say here is going to be a lie on some platforms.
>
> Probably best just to say that the sleep resolution is platform-dependent
> and leave it at that.
I think, we can say "on many systems"/ "on most Unixen", and it would
not be a lie.
In my opinion, while a generic reference to platform-dependency is OK
for developer' documentation, it makes the following passage in
config.sgml vague (we don't give user a hint, what "the effective
resolution" can be - several seconds/milliseconds/nanoseconds?):
/The default value is 200 milliseconds (<literal>200ms</literal>). Note
that on many systems, the//
//effective resolution of sleep delays is 10 milliseconds; setting//
//<varname>bgwriter_delay</varname> to a value that is not a multiple of
10//
//might have the same results as setting it to the next higher multiple
of 10. /
->
/The default value is 200 milliseconds (<literal>200ms</literal>). Note
that the//
//effective resolution of sleep delays is paltform-dependent. setting//
//<varname>bgwriter_delay</varname> to a value that is not a multiple of
the effective resolution,/
/might have the same results as setting it to the next higher multiple./

Best regards,
Alexander

--------------DEAE41C3D7AA538074018D8C
Content-Type: text/html; charset=utf-8
Content-Transfer-Encoding: 7bit

<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">Hello Tom,<br>
      22.07.2019 7:14, Tom Lane wrote:<br>
    </div>
    <blockquote type="cite" cite="mid:28139.1563768879@sss.pgh.pa.us">
      <blockquote type="cite">
        <pre class="moz-quote-pre" wrap="">Fixing both places sounds adapted to me.  An alternative we could use
here is just to say something like that:
The effective resolution is only 1/HZ, which can be configured with
kernel parameter (see man 7 time), and is 4 milliseconds by
default.
</pre>
      </blockquote>
      <pre class="moz-quote-pre" wrap="">Whatever we say here is going to be a lie on some platforms.

Probably best just to say that the sleep resolution is platform-dependent
and leave it at that.
</pre>
    </blockquote>
    I think, we can say "on many systems"/ "on most Unixen", and it
    would not be a lie.<br>
    In my opinion, while a generic reference to platform-dependency is
    OK for developer' documentation, it makes the following passage in
    config.sgml vague (we don't give user a hint, what "the effective
    resolution" can be - several seconds/milliseconds/nanoseconds?):<br>
    <i>The default value is 200 milliseconds
      (&lt;literal&gt;200ms&lt;/literal&gt;). Note that on many systems,
      the</i><i><br>
    </i><i>effective resolution of sleep delays is 10 milliseconds;
      setting</i><i><br>
    </i><i>&lt;varname&gt;bgwriter_delay&lt;/varname&gt; to a value that
      is not a multiple of 10</i><i><br>
    </i><i>might have the same results as setting it to the next higher
      multiple of 10. </i><br>
    -&gt;<br>
    <i>The default value is 200 milliseconds
      (&lt;literal&gt;200ms&lt;/literal&gt;). Note that the</i><i><br>
    </i><i>
      effective resolution of sleep delays is paltform-dependent.
      setting</i><i><br>
    </i><i>
      &lt;varname&gt;bgwriter_delay&lt;/varname&gt; to a value that is
      not a multiple of the effective resolution,</i><br>
    <i>might have the same results as setting it to the next higher
      multiple.</i><br>
    <br>
    Best regards,<br>
    Alexander<br>
  </body>
</html>

--------------DEAE41C3D7AA538074018D8C--