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.

-- 
Álvaro Herrera               48°01'N 7°57'E  —  https://www.EnterpriseDB.com/


Reply via email to