Horton

Re: Documenting MERGE INTO ONLY ...

Dean Rasheed <dean.a.rasheed@gmail.com>

View thread View on postgr.es
From: Dean Rasheed <dean.a.rasheed@gmail.com>
To: Alvaro Herrera <alvherre@alvh.no-ip.org>
Cc: Nathan Bossart <nathandbossart@gmail.com>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2022-12-09T10:09:38Z
Lists: pgsql-hackers 
On Fri, 9 Dec 2022 at 10:02, Alvaro Herrera <alvherre@alvh.no-ip.org> wrote:
>
> On 2022-Dec-08, Nathan Bossart wrote:
>
> > On Thu, Dec 08, 2022 at 03:26:52PM +0000, Dean Rasheed wrote:
> > > While testing MERGE, I noticed that it supports inheritance
> > > hierarchies and the ONLY keyword, but that isn't documented. Attached
> > > is a patch to merge.sgml, borrowing text from update.sgml and
> > > delete.sgml.
> >
> > LGTM.
>
> LGTM2.
>
> > I didn't see any tests for this in merge.sql or inherit.sql.  Do you
> > think we should add some?
>
> Ouch!  We should definitely have some.
>

Agreed. I just pushed this, with some additional tests.

Regards,
Dean

Commits

  1. Update MERGE docs to mention that ONLY is supported.