From: Tom Lane <tgl@sss.pgh.pa.us>
To: Amit Langote <amitlangote09@gmail.com>
cc: Alvaro Herrera <alvherre@2ndquadrant.com>, Peter Geoghegan <pg@bowt.ie>,
        PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Subject: Re: Fixing findDependentObjects()'s dependency on scan order
 (regressions in DROP diagnostic messages)
In-reply-to: 
 <CA+HiwqE+qiSy1JLS63NhpLPRO4dpO6nfK8ZyjK2V-Pz3FCCnEQ@mail.gmail.com>
References: <5943.1549728638@sss.pgh.pa.us>
 <20190209165010.GA4298@alvherre.pgsql>
 <CA+HiwqE+qiSy1JLS63NhpLPRO4dpO6nfK8ZyjK2V-Pz3FCCnEQ@mail.gmail.com>
Comments: In-reply-to Amit Langote <amitlangote09@gmail.com>
	message dated "Sun, 10 Feb 2019 02:06:47 +0900"
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-ID: <24134.1549732381.1@sss.pgh.pa.us>
Date: Sat, 09 Feb 2019 12:13:01 -0500
Message-ID: <24135.1549732381@sss.pgh.pa.us>
Precedence: bulk

Amit Langote <amitlangote09@gmail.com> writes:
> Reading Tom's reply to my email, I wondered if performDeletion won't
> do more than what the code is already doing (except calling the right
> trigger deletion function which the current code doesn't), because the
> trigger in question is an internal trigger without any dependencies
> (the function it invokes are pinned by the system)?

A big part of the point here is to not have to have such assumptions
wired into the fk-cloning code.  But even if that internal dependency is
the only one the trigger is involved in, there are other steps in
deleteOneObject that shouldn't be ignored.  For example, somebody
could've attached a comment to it.

			regards, tom lane