Re: bgwriter_lru_multiplier maximum value

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(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Michael Paquier <michael(at)paquier(dot)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
>