Thread: Fix the README file for MERGE command
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
Attachment
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.
Thanks
Richard
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 W. <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> 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
> 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/