Amit-san, On Wed, Jun 12, 2019 at 3:33 PM Amit Langote <amitlangot...@gmail.com> wrote: > On Wed, Jun 12, 2019 at 3:14 PM Etsuro Fujita <etsuro.fuj...@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
