On Fri, Aug 14, 2020 at 10:29 AM Thomas Munro <thomas.munro@gmail.com> wrote:
> On Thu, Jul 2, 2020 at 3:20 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> > I'd like to join the party, but IIUC, we don't yet reach a consensus
> > on which one is the right way to go. So I think we need to discuss
> > that first.
>
> Either way, we definitely need patch 0001. One comment:
>
> -CreateWaitEventSet(MemoryContext context, int nevents)
> +CreateWaitEventSet(MemoryContext context, ResourceOwner res, int nevents)
>
> I wonder if it's better to have it receive ResourceOwner like that, or
> to have it capture CurrentResourceOwner. I think the latter is more
> common in existing code.
Sorry for not having discussed anything, but actually, I’ve started
reviewing your patch first. I’ll return to this after reviewing it
some more.
Thanks!
Best regards,
Etsuro Fujita