Horton

Thread

Commits

  1. doc: Fix wording of MERGE actions in README

  1. Fix the README file for MERGE command

    Waithant Myo (Fujitsu) <myo.waithant@fujitsu.com> — 2022-11-16T08:37:12Z

    Hi Hackers,

This is Myo Wai Thant.
I found out that there is a mistake written in executor/README file.

The actions of MERGE command can be specified as follows: INSERT, UPDATE, DELETE and DO NOTHING.
However, in the README file, the ‘UPDATE’ word is described 2 times instead of ‘DELETE’.

Therefore, I attached the patch file which fix this word usage.
It would be great if you could take a look at it.

Thank you.
Best Regards,
Myo Wai Thant

  2. Re: Fix the README file for MERGE command

    Richard Guo <guofenglinux@gmail.com> — 2022-11-17T01:31:01Z

    On Wed, Nov 16, 2022 at 4:37 PM Waithant Myo (Fujitsu) <
myo.waithant@fujitsu.com> wrote:

> The actions of MERGE command can be specified as follows: INSERT, UPDATE,
> DELETE and DO NOTHING.
>
> However, in the README file, the ‘UPDATE’ word is described 2 times
> instead of ‘DELETE’.
>
>
>
> Therefore, I attached the patch file which fix this word usage.
>
> It would be great if you could take a look at it.
>

Apparently this is a typo.  Good catch! +1.

Thanks
Richard

  3. RE: Fix the README file for MERGE command

    Waithant Myo (Fujitsu) <myo.waithant@fujitsu.com> — 2022-11-17T05:16:47Z

    Hi Richard,

Thank you for your time.

Best Regards,
Myo Wai Thant
From: Richard Guo <guofenglinux@gmail.com>
Sent: Thursday, November 17, 2022 10:31 AM
To: Myo, Waithant/Ｍｙｏ Ｗ． <myo.waithant@fujitsu.com>
Cc: pgsql-hackers@lists.postgresql.org
Subject: Re: Fix the README file for MERGE command


On Wed, Nov 16, 2022 at 4:37 PM Waithant Myo (Fujitsu) <myo.waithant@fujitsu.com<mailto:myo.waithant@fujitsu.com>> wrote:
The actions of MERGE command can be specified as follows: INSERT, UPDATE, DELETE and DO NOTHING.
However, in the README file, the ‘UPDATE’ word is described 2 times instead of ‘DELETE’.

Therefore, I attached the patch file which fix this word usage.
It would be great if you could take a look at it.

Apparently this is a typo.  Good catch! +1.

Thanks
Richard

  4. Re: Fix the README file for MERGE command

    Daniel Gustafsson <daniel@yesql.se> — 2022-11-17T09:12:34Z

    > On 17 Nov 2022, at 02:31, Richard Guo <guofenglinux@gmail.com> wrote:
> On Wed, Nov 16, 2022 at 4:37 PM Waithant Myo (Fujitsu) <myo.waithant@fujitsu.com <mailto:myo.waithant@fujitsu.com>> wrote: 
> Therefore, I attached the patch file which fix this word usage.
> 
> It would be great if you could take a look at it.
> 
> Apparently this is a typo.  Good catch! +1.


Agreed. I've applied this down to v15 where MERGE was introduced. Thanks!

--
Daniel Gustafsson		https://vmware.com/