MIME-Version: 1.0
From: Fujii Masao <masao.fujii@gmail.com>
Date: Fri, 10 Apr 2026 12:00:38 +0900
Message-ID: 
 <CAHGQGwGnCVMVz8-LU9F8Sh57bkQX3jMZzx7age7M0LFEz5=Fog@mail.gmail.com>
Subject: Clarify log_parameter_max_length docs regarding
 log_min_duration_statement
 vs log_duration
To: pgsql-docs@lists.postgresql.org
Content-Type: multipart/mixed; boundary="000000000000385c5a064f125797"
Archived-At: 
 <https://www.postgresql.org/message-id/CAHGQGwGnCVMVz8-LU9F8Sh57bkQX3jMZzx7age7M0LFEz5%3DFog%40mail.gmail.com>
Precedence: bulk

--000000000000385c5a064f125797
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi,

While reviewing the log_statement_max_length patch [1], I noticed the docs
for log_parameter_max_length say:

        This setting only affects log messages printed as a result of
        log_statement, log_duration, and related settings.=E2=80=9D

However, enabling only log_duration does not log bind parameter values,
so in practice log_parameter_max_length does not seem to affect messages
generated by log_duration. Bind parameters are logged, for example, when
log_min_duration_statement is set.

Given that, referencing log_duration here seems misleading. Would it be
better to mention log_min_duration_statement instead?

I've attached a patch that makes this change.

Regards,


[1] https://postgr.es/m/CA+E0NR4S+NC6+QHyY_vUuQZMzLhKqczMx-jJVqtjAxF6+=3DJw=
AA@mail.gmail.com


--=20
Fujii Masao

--000000000000385c5a064f125797
Content-Type: application/octet-stream;
	name="v1-0001-doc-Fix-log_parameter_max_length-docs-to-referenc.patch"
Content-Disposition: attachment;
	filename="v1-0001-doc-Fix-log_parameter_max_length-docs-to-referenc.patch"
Content-Transfer-Encoding: base64
Content-ID: <f_mnsbhxsa0>
X-Attachment-Id: f_mnsbhxsa0

RnJvbSBjYzU0MTk2NzliNTViOWIzNzUxMDVkODg2NTdlZjUyZDQxZGVhM2ZmIE1vbiBTZXAgMTcg
MDA6MDA6MDAgMjAwMQpGcm9tOiBGdWppaSBNYXNhbyA8ZnVqaWlAcG9zdGdyZXNxbC5vcmc+CkRh
dGU6IEZyaSwgMTAgQXByIDIwMjYgMTE6MDQ6MjIgKzA5MDAKU3ViamVjdDogW1BBVENIIHYxXSBk
b2M6IEZpeCBsb2dfcGFyYW1ldGVyX21heF9sZW5ndGggZG9jcyB0byByZWZlcmVuY2UKIGxvZ19t
aW5fZHVyYXRpb25fc3RhdGVtZW50CgpUaGUgZG9jdW1lbnRhdGlvbiBmb3IgbG9nX3BhcmFtZXRl
cl9tYXhfbGVuZ3RoIHNhaWQgaXQgYWZmZWN0cyBtZXNzYWdlcwpnZW5lcmF0ZWQgYnkgbG9nX2R1
cmF0aW9uLiBIb3dldmVyLCBsb2dfZHVyYXRpb24gYWxvbmUgZG9lcyBub3QgbG9nIGJpbmQKcGFy
YW1ldGVyIHZhbHVlcywgc28gdGhpcyBpcyBtaXNsZWFkaW5nLgoKVGhpcyBjb21taXQgdXBkYXRl
cyB0aGUgZG9jdW1lbnRhdGlvbiB0byByZWZlcmVuY2UgbG9nX21pbl9kdXJhdGlvbl9zdGF0ZW1l
bnQsCndoaWNoIGNhbiBsb2cgYmluZCBwYXJhbWV0ZXJzLCB0byBiZXR0ZXIgcmVmbGVjdCBhY3R1
YWwgYmVoYXZpb3IuCi0tLQogZG9jL3NyYy9zZ21sL2NvbmZpZy5zZ21sIHwgNyArKysrLS0tCiAx
IGZpbGUgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCspLCAzIGRlbGV0aW9ucygtKQoKZGlmZiAtLWdp
dCBhL2RvYy9zcmMvc2dtbC9jb25maWcuc2dtbCBiL2RvYy9zcmMvc2dtbC9jb25maWcuc2dtbApp
bmRleCAwMDkyOTVmMjBlOS4uODhlMWM1OGYzNTIgMTAwNjQ0Ci0tLSBhL2RvYy9zcmMvc2dtbC9j
b25maWcuc2dtbAorKysgYi9kb2Mvc3JjL3NnbWwvY29uZmlnLnNnbWwKQEAgLTgzODgsOSArODM4
OCwxMCBAQCBsb2dfbGluZV9wcmVmaXggPSAnJW0gWyVwXSAlcSV1QCVkLyVhICcKICAgICAgICA8
cGFyYT4KICAgICAgICAgVGhpcyBzZXR0aW5nIG9ubHkgYWZmZWN0cyBsb2cgbWVzc2FnZXMgcHJp
bnRlZCBhcyBhIHJlc3VsdCBvZgogICAgICAgICA8eHJlZiBsaW5rZW5kPSJndWMtbG9nLXN0YXRl
bWVudCIvPiwKLSAgICAgICAgPHhyZWYgbGlua2VuZD0iZ3VjLWxvZy1kdXJhdGlvbiIvPiwgYW5k
IHJlbGF0ZWQgc2V0dGluZ3MuICBOb24temVybwotICAgICAgICB2YWx1ZXMgb2YgdGhpcyBzZXR0
aW5nIGFkZCBzb21lIG92ZXJoZWFkLCBwYXJ0aWN1bGFybHkgaWYgcGFyYW1ldGVycwotICAgICAg
ICBhcmUgc2VudCBpbiBiaW5hcnkgZm9ybSwgc2luY2UgdGhlbiBjb252ZXJzaW9uIHRvIHRleHQg
aXMgcmVxdWlyZWQuCisgICAgICAgIDx4cmVmIGxpbmtlbmQ9Imd1Yy1sb2ctbWluLWR1cmF0aW9u
LXN0YXRlbWVudCIvPiwgYW5kIHJlbGF0ZWQgc2V0dGluZ3MuCisgICAgICAgIE5vbi16ZXJvIHZh
bHVlcyBvZiB0aGlzIHNldHRpbmcgYWRkIHNvbWUgb3ZlcmhlYWQsIHBhcnRpY3VsYXJseQorICAg
ICAgICBpZiBwYXJhbWV0ZXJzIGFyZSBzZW50IGluIGJpbmFyeSBmb3JtLCBzaW5jZSB0aGVuIGNv
bnZlcnNpb24gdG8KKyAgICAgICAgdGV4dCBpcyByZXF1aXJlZC4KICAgICAgICA8L3BhcmE+CiAg
ICAgICA8L2xpc3RpdGVtPgogICAgICA8L3Zhcmxpc3RlbnRyeT4KLS0gCjIuNTEuMgoK
--000000000000385c5a064f125797--