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

On Mon, Sep 28, 2026 at 12:51 AM shihao zhong <zhong950419@gmail.com> wrote:
>
> Hi Andrew,
>
> > 1) I think 0001 is not needed anymore once 0002 is applied. With
> > 0002, the waiting backend always clears waitStart itself when the wait
> > ends at the end of ProcSleep(), or in LockErrorCleanup().
> > RemoveFromWaitQueue() is only called from CheckDeadLock() and
> > LockErrorCleanup(), and both paths reach one of these new resets.
> >
> > Maybe it is simpler to merge 0001 and 0002 into one commit?
> > They fix the same problem and would be backpatched together.
>
> Agreed, v3 merges them into one patch. The code is the same as v2.

Thanks for updating the patch! I made only cosmetic changes and pushed it.

Regards,

--
Fujii Masao