Re: Some comments that should've covered MERGE

2022-10-24 Thread Richard Guo
On Mon, Oct 24, 2022 at 6:59 PM Alvaro Herrera wrote: > On 2022-Oct-19, Richard Guo wrote: > > > Hi hackers, > > > > I happened to notice $subject. Attach a trivial patch for that. > > Thanks, applied. I did change the comment atop setTargetTable, which I > thought could use a little bit more de

Re: Some comments that should've covered MERGE

2022-10-24 Thread Alvaro Herrera
On 2022-Oct-19, Richard Guo wrote: > Hi hackers, > > I happened to notice $subject. Attach a trivial patch for that. Thanks, applied. I did change the comment atop setTargetTable, which I thought could use a little bit more detail on what is happening, and also in its callsite in transformMerge