From: | Amit Langote <amitlangote09(at)gmail(dot)com> |
---|---|
To: | Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: Inherited UPDATE/DELETE vs async execution |
Date: | 2021-05-13 12:09:33 |
Message-ID: | CA+HiwqG6fjj5FJw=A7zUDFgTSHg9B+k+bkdLMNqAUVEBOKD0Eg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, May 13, 2021 at 8:10 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> On Thu, May 13, 2021 at 5:00 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)com> wrote:
> > On Thu, May 13, 2021 at 3:32 PM Amit Langote <amitlangote09(at)gmail(dot)com> wrote:
> > > On Wed, May 12, 2021 at 6:45 PM Etsuro Fujita <etsuro(dot)fujita(at)gmail(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | vignesh C | 2021-05-13 12:11:43 | Re: subscriptioncheck failure |
Previous Message | Justin Pryzby | 2021-05-13 12:06:55 | Re: Teaching users how they can get the most out of HOT in Postgres 14 |