pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
bgwriter_lru_multiplier maximum value
5+ messages / 4 participants
[nested] [flat]

* bgwriter_lru_multiplier maximum value
@ 2019-04-01 10:27  PG Doc comments form <noreply@postgresql.org>
  0 siblings, 1 reply; 5+ messages in thread

From: PG Doc comments form @ 2019-04-01 10:27 UTC (permalink / raw)
  To: pgsql-docs@lists.postgresql.org; +Cc: chris+postgresql@qwirx.com

The following documentation comment has been logged on the website:

Page: https://www.postgresql.org/docs/11/runtime-config-resource.html
Description:

The server has an undocumented maximum value of 10 for
bgwriter_lru_multiplier, and will fail to start if it's set higher than
this:

LOG:  20 is outside the valid range for parameter "bgwriter_lru_multiplier"
(0 .. 10)
FATAL:  configuration file "/.../pgdata/postgresql.conf" contains errors


^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: bgwriter_lru_multiplier maximum value
@ 2019-04-01 13:18  Michael Paquier <michael@paquier.xyz>
  parent: PG Doc comments form <noreply@postgresql.org>
  0 siblings, 1 reply; 5+ messages in thread

From: Michael Paquier @ 2019-04-01 13:18 UTC (permalink / raw)
  To: chris+postgresql@qwirx.com; pgsql-docs@lists.postgresql.org

On Mon, Apr 01, 2019 at 10:27:13AM +0000, PG Doc comments form wrote:
> The server has an undocumented maximum value of 10 for
> bgwriter_lru_multiplier, and will fail to start if it's set higher than
> this:
> 
> LOG:  20 is outside the valid range for parameter "bgwriter_lru_multiplier"
> (0 .. 10)
> FATAL:  configuration file "/.../pgdata/postgresql.conf" contains errors

I would suggest something like that:
inline patch 1: doc/src/sgml/config.sgml (+2/-1 lines) download inline-1.patch--- a/doc/src/sgml/config.sgml +++ b/doc/src/sgml/config.sgml @@ -2050,7 +2050,8 @@ include_dir 'conf.d' Larger values provide some cushion against spikes in demand, while smaller values intentionally leave writes to be done by server processes. - The default is 2.0. + The default is 2.0. The value can range from <literal>2.0</literal> + to <literal>10.0</literal> This parameter can only be set in the <filename>postgresql.conf</filename> file or on the server command line. </para>
Any opinions from others? -- Michael Attachments: signature.asc (application/pgp-signature, 832B) download ^ permalink raw reply [nested|flat] 5+ messages in thread

* Re: bgwriter_lru_multiplier maximum value
@ 2019-04-01 13:22  Tom Lane <tgl@sss.pgh.pa.us>
  parent: Michael Paquier <michael@paquier.xyz>
  0 siblings, 1 reply; 5+ messages in thread

From: Tom Lane @ 2019-04-01 13:22 UTC (permalink / raw)
  To: Michael Paquier <michael@paquier.xyz>; +Cc: chris+postgresql@qwirx.com; pgsql-docs@lists.postgresql.org

Michael Paquier <michael@paquier.xyz> writes:
> On Mon, Apr 01, 2019 at 10:27:13AM +0000, PG Doc comments form wrote:
>> The server has an undocumented maximum value of 10 for
>> bgwriter_lru_multiplier, and will fail to start if it's set higher than
>> this:

> Any opinions from others?

I don't think it's generally our practice to specify min and max
values in config.sgml.  If we were to start doing so, we'd need
a much bigger patch than this, and it'd be an ongoing maintenance
issue.

Unless there's some reason I'm missing why the limits are more
significant for this variable than for others?

			regards, tom lane





^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: bgwriter_lru_multiplier maximum value
@ 2019-04-01 14:09  Chris Wilson <chris+postgresql@qwirx.com>
  parent: Tom Lane <tgl@sss.pgh.pa.us>
  0 siblings, 1 reply; 5+ messages in thread

From: Chris Wilson @ 2019-04-01 14:09 UTC (permalink / raw)
  To: Tom Lane <tgl@sss.pgh.pa.us>; +Cc: Michael Paquier <michael@paquier.xyz>; pgsql-docs@lists.postgresql.org

Hi all,

We do document the range for some parameters:

   - bgwriter_flush_after: "The valid range is between 0, which disables
   forced writeback, and 2MB."
   - effective_io_concurrency (integer): "The allowed range is 1 to 1000,
   or zero to disable issuance of asynchronous I/O requests."
   - backend_flush_after (integer): "The valid range is between 0, which
   disables forced writeback, and 2MB."

My argument for documenting this one would extend to all parameters that
can only be set at server startup time, and for which exceeding the maximum
(or minimum) value would cause the server to fail to start. In these cases
the change cannot be tested without risk of downtime. It's also not clear
where to look (e..g in the code) for the limits on allowed range.

+         The default is 2.0.  The value can range from
> <literal>2.0</literal>


If we are going to change it, the minimum value is apparently 0, not 2.

Thanks, Chris.

On Mon, 1 Apr 2019 at 14:23, Tom Lane <tgl@sss.pgh.pa.us> wrote:

> Michael Paquier <michael@paquier.xyz> writes:
> > On Mon, Apr 01, 2019 at 10:27:13AM +0000, PG Doc comments form wrote:
> >> The server has an undocumented maximum value of 10 for
> >> bgwriter_lru_multiplier, and will fail to start if it's set higher than
> >> this:
>
> > Any opinions from others?
>
> I don't think it's generally our practice to specify min and max
> values in config.sgml.  If we were to start doing so, we'd need
> a much bigger patch than this, and it'd be an ongoing maintenance
> issue.
>
> Unless there's some reason I'm missing why the limits are more
> significant for this variable than for others?
>
>                         regards, tom lane
>

^ permalink  raw  reply  [nested|flat] 5+ messages in thread

* Re: bgwriter_lru_multiplier maximum value
@ 2019-04-02 01:52  Michael Paquier <michael@paquier.xyz>
  parent: Chris Wilson <chris+postgresql@qwirx.com>
  0 siblings, 0 replies; 5+ messages in thread

From: Michael Paquier @ 2019-04-02 01:52 UTC (permalink / raw)
  To: Chris Wilson <chris+postgresql@qwirx.com>; +Cc: Tom Lane <tgl@sss.pgh.pa.us>; pgsql-docs@lists.postgresql.org

On Mon, Apr 01, 2019 at 03:09:39PM +0100, Chris Wilson wrote:
> If we are going to change it, the minimum value is apparently 0, not
> 2.

Ditto.  I misread guc.c.
--
Michael

Attachments:

  signature.asc (application/pgp-signature, 832B) download

^ permalink  raw  reply  [nested|flat] 5+ messages in thread


end of thread, other threads:[~2019-04-02 01:52 UTC | newest]

Thread overview: 5+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2019-04-01 10:27 bgwriter_lru_multiplier maximum value PG Doc comments form <noreply@postgresql.org>
2019-04-01 13:18 ` Michael Paquier <michael@paquier.xyz>
2019-04-01 13:22   ` Tom Lane <tgl@sss.pgh.pa.us>
2019-04-01 14:09     ` Chris Wilson <chris+postgresql@qwirx.com>
2019-04-02 01:52       ` Michael Paquier <michael@paquier.xyz>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox