Horton

Thread

Commits

  1. Update some comments that should've covered MERGE

  1. Some comments that should've covered MERGE

    Richard Guo <guofenglinux@gmail.com> — 2022-10-19T08:02:34Z

    Hi hackers,

I happened to notice $subject. Attach a trivial patch for that.

Thanks
Richard

  2. Re: Some comments that should've covered MERGE

    Alvaro Herrera <alvherre@alvh.no-ip.org> — 2022-10-24T10:59:06Z

    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 transformMergeStmt.

-- 
Álvaro Herrera         PostgreSQL Developer  —  https://www.EnterpriseDB.com/
"La experiencia nos dice que el hombre peló millones de veces las patatas,
pero era forzoso admitir la posibilidad de que en un caso entre millones,
las patatas pelarían al hombre" (Ijon Tichy)

  3. Re: Some comments that should've covered MERGE

    Richard Guo <guofenglinux@gmail.com> — 2022-10-25T02:15:18Z

    On Mon, Oct 24, 2022 at 6:59 PM Alvaro Herrera <alvherre@alvh.no-ip.org>
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 detail on what is happening, and
> also in its callsite in transformMergeStmt.


Thanks for the fix!

Thanks
Richard