Amit-san,
On Wed, Jun 12, 2019 at 3:33 PM Amit Langote <amitlangote09@gmail.com> wrote:
> On Wed, Jun 12, 2019 at 3:14 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> > * Reworded the comments a bit in postgresPlanFoereignModify the
> > original patch modified
>
> + * statement, and for UPDATE if BEFORE ROW UPDATE triggers since those
> + * triggers might change values for non-target columns, in which case we
>
> First line seems to be missing a word or two. Maybe:
>
> + * statement, and for UPDATE if there are BEFORE ROW UPDATE triggers,
> + * since those triggers might change values for non-target columns, in
Actually, I omitted such words to shorten the comment, but I think
this improves the readability, so I'll update the comment that way.
Thanks for the review!
Best regards,
Etsuro Fujita