Re: Reset waitStart when a lock wait fails - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: Reset waitStart when a lock wait fails
Date
Msg-id arSwgp1VLVKpRTV5@paquier.xyz
Whole thread
In response to Re: Reset waitStart when a lock wait fails  (Chao Li <li.evan.chao@gmail.com>)
Responses Re: Reset waitStart when a lock wait fails
List pgsql-hackers
On Thu, Sep 24, 2026 at 10:37:12AM +0800, Chao Li wrote:
> On Sep 24, 2026, at 09:30, shihao zhong <zhong950419@gmail.com> wrote:
>> With the patch it reads NULL. The patch clears it the same way
>> ProcWakeup() does, like 70f470314cb did for the grant path. It applies
>> to 14 and up.
>
> Looks good to me. Once proc->waitLock is cleared, proc->waitStart no
> longer describes an active wait, so clearing it at the same time
> seems right.

Yep, it looks weird to not clean up the field in this code path.
@Fujii-san, what do you think?
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: "Hayato Kuroda (Fujitsu)"
Date:
Subject: RE: [PATCH] Explain what the default output_plugin_libraries do
Next
From: "Hayato Kuroda (Fujitsu)"
Date:
Subject: RE: [PATCH] Explain what the default output_plugin_libraries do