Re: Inherited UPDATE/DELETE vs async execution

On Thu, May 13, 2021 at 8:10 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> On Thu, May 13, 2021 at 5:00 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> > On Thu, May 13, 2021 at 3:32 PM Amit Langote <amitlangote09@gmail.com> wrote:
> > > On Wed, May 12, 2021 at 6:45 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> > > > Here is a rebased version of the patch.  I'm planning to apply this tommorow.
> > >
> > > +   /*
> > > +    * Finally, unset the async-capable flag if it is set.
> > > +    */
> > >
> > > Would it make sense to expand here even just a bit on why we must do this?
> >
> > +1  How about something like this?
> >
> > "Finally, unset the async-capable flag if it is set, as we currently
> > don't support asynchronous execution of direct modifications."
>
> Pushed after modifying the comment as such.  I think we could improve
> it later.  :-)

Looks good as pushed, thank you.

-- 
Amit Langote
EDB: http://www.enterprisedb.com