Re: Add new wait event to XactLockTableWait

On Mon, Jun 9, 2025 at 2:18 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>
>
>
> On 2025/06/09 14:19, Xuneng Zhou wrote:
> > Hi,
> >
> > On Mon, Jun 9, 2025 at 12:52 PM Xuneng Zhou <xunengzhou@gmail.com <mailto:xunengzhou@gmail.com>> wrote:
> >  >
> >  > Hi,
> >  >
> >  > > Please find attached Version 2, incorporating the suggested changes.
> >  >
> >  > Apologies for the confusion — in the previous attempt, I mistakenly
> >  > named the patch file with a `0002-` prefix, thinking it reflected the
> >  > patch version rather than the patch series number.  I've corrected the
> >  > filename to follow the proper convention.
> >
> > Sorry for the missing hyphen in the patch name. Things should be in good shape now. I need to be more cautious and
double-checkeverything before submitting patches and sending emails.😂 
>
> Thanks for updating the patch! It looks good to me.
>
> I think we can mark it as "Ready for Committer" in the CommitFest.
> Unless there are any objections, I'll commit it once v19 development opens.

LGTM, except I suggest using WAIT_EVENT_XACT_COMPLETE instead of
WAIT_EVENT_XACT_DONE. I think it sounds better.

--
Regards,
Dilip Kumar
Google