Hi
On 2020-03-12 21:34, Bruce Momjian wrote:
> On Mon, Feb 3, 2020 at 05:04:00PM +0000, PG Doc comments form wrote:
>> The following documentation comment has been logged on the website:
>>
>> Page: https://www.postgresql.org/docs/9.4/explicit-locking.html
>> Description:
>>
>> Hi
>>
>> The "13.3.1. Table-level Locks" subsection mentions the following:
>> "Once
>> acquired, a lock is normally held till end of transaction." (maybe we
>> should
>> also squeeze a "...till the end of a transaction" in there) According
>> to a
>
> Sorry for the delay in replying. Yes, this wording needs improvement,
No problem!
> which I have done in the attached patch.
>
>> helpful stranger on IRC, this behavior is also true for row-level
>> locks.
>>
>> Since this sentence also applies to the row-level locks described in
>> the
>> following subsection 13.3.2 I think it would be more fitting to move
>> the
>> paragraph containing this sentence to the introduction of the topic in
>> section "13.3. Explicit Locking". This would then read something like:
>
> Uh, we can't move that paragraph up because Page-Level Locks and
> Advisory Locks aren't always released on transaction end or rollback.
Ah, I didn't take that into account.
> What I did do was to mention that row-level locks are released in a
> similar way to table-level locks.
>
> Patch attached.
The patch looks good to me, thanks!
Cheers,
Silvan