pg.ddx.io pgsql-docs@postgresql.org mailing list archive
help / color / mirror / Atom feedA Rant on lock_timeout
2+ messages / 2 participants
[nested] [flat]
Commits from this thread:
2019-10-07 47571ec1e4 6fd03a95eb f872f18173 ef0668a51c cf3187e0e0 873cd4f606 7eefec61c9 doc: move mention of log_min_error_statement in a better spot
* A Rant on lock_timeout
@ 2019-10-04 13:35 PG Doc comments form <noreply@postgresql.org>
0 siblings, 1 reply; 2+ messages in thread
From: PG Doc comments form @ 2019-10-04 13:35 UTC (permalink / raw)
To: pgsql-docs@lists.postgresql.org; +Cc: ivaylo.zlatanov@gmail.com
The following documentation comment has been logged on the website:
Page: https://www.postgresql.org/docs/12/runtime-config-client.html
Description:
Hello,
I was looking explicitly for information on the default value of
lock_timeout and what it does. I got this paragraph:
"Abort any statement that waits longer than the specified number of
milliseconds while attempting to acquire a lock on a table, index, row, or
other database object. The time limit applies separately to each lock
acquisition attempt. The limit applies both to explicit locking requests
(such as LOCK TABLE, or SELECT FOR UPDATE without NOWAIT) and to
implicitly-acquired locks. If log_min_error_statement is set to ERROR or
lower, the statement that timed out will be logged. A value of zero (the
default) turns this off."
Unfortunately, the crucial bit for me is in a sentence that hardly could be
any more obscure: `A value of zero (the default) turns this off.` Before
this sentence there are multiple statements; what exactly does `this` refers
to? Does a value of zero turns of logging of timed out statements mentioned
in the previous sentence? Would it be hard to put it like `A default value
of zero means a statement will wait indefinitely to acquire a lock`.
I believe the sole purpose of a technical document is to be precise. If I
had to suggest setting this parameter as a solution to a critical issue of
connections piling up waiting on locks and depleting connection pools, I can
never be confident knowing that the current default value turns something
off.
^ permalink raw reply [nested|flat] 2+ messages in thread
* Re: A Rant on lock_timeout
@ 2019-10-07 19:19 Bruce Momjian <bruce@momjian.us>
parent: PG Doc comments form <noreply@postgresql.org>
0 siblings, 0 replies; 2+ messages in thread
From: Bruce Momjian @ 2019-10-07 19:19 UTC (permalink / raw)
To: ivaylo.zlatanov@gmail.com; pgsql-docs@lists.postgresql.org
On Fri, Oct 4, 2019 at 01:35:57PM +0000, PG Doc comments form wrote:
> The following documentation comment has been logged on the website:
>
> Page: https://www.postgresql.org/docs/12/runtime-config-client.html
> Description:
>
> Hello,
>
> I was looking explicitly for information on the default value of
> lock_timeout and what it does. I got this paragraph:
>
> "Abort any statement that waits longer than the specified number of
> milliseconds while attempting to acquire a lock on a table, index, row, or
> other database object. The time limit applies separately to each lock
> acquisition attempt. The limit applies both to explicit locking requests
> (such as LOCK TABLE, or SELECT FOR UPDATE without NOWAIT) and to
> implicitly-acquired locks. If log_min_error_statement is set to ERROR or
> lower, the statement that timed out will be logged. A value of zero (the
> default) turns this off."
>
> Unfortunately, the crucial bit for me is in a sentence that hardly could be
> any more obscure: `A value of zero (the default) turns this off.` Before
> this sentence there are multiple statements; what exactly does `this` refers
> to? Does a value of zero turns of logging of timed out statements mentioned
> in the previous sentence? Would it be hard to put it like `A default value
> of zero means a statement will wait indefinitely to acquire a lock`.
> I believe the sole purpose of a technical document is to be precise. If I
> had to suggest setting this parameter as a solution to a critical issue of
> connections piling up waiting on locks and depleting connection pools, I can
> never be confident knowing that the current default value turns something
> off.
I completely agree. I have applied the attached patch to all supported
doc versions to move the mention of log_min_error_statement to a more
logical location. Thanks for the report.
--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ As you are, so once was I. As I am, so you will be. +
+ Ancient Roman grave inscription +
^ permalink raw reply [nested|flat] 2+ messages in thread
end of thread, other threads:[~2019-10-07 19:19 UTC | newest]
Thread overview: 2+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2019-10-04 13:35 A Rant on lock_timeout PG Doc comments form <noreply@postgresql.org>
2019-10-07 19:19 ` Bruce Momjian <bruce@momjian.us>
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