pgsql: Create foreign key triggers in partitioned tables too

From: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Create foreign key triggers in partitioned tables too
Date: 2022-01-05 22:15:56
Message-ID: E1n5EZY-0004Yx-SX@gemulon.postgresql.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Create foreign key triggers in partitioned tables too

While user-defined triggers defined on a partitioned table have
a catalog definition for both it and its partitions, internal
triggers used by foreign keys defined on partitioned tables only
have a catalog definition for its partitions. This commit fixes
that so that partitioned tables get the foreign key triggers too,
just like user-defined triggers. Moreover, like user-defined
triggers, partitions' internal triggers will now also have their
tgparentid set appropriately. This is to allow subsequent commit(s)
to make the foreign key related events to be fired in some cases
using the parent table triggers instead of those of partitions'.

This also changes what tgisinternal means in some cases. Currently,
it means either that the trigger is an internal implementation object
of a foreign key constraint, or a "child" trigger on a partition
cloned from the trigger on the parent. This commit changes it to
only mean the former to avoid confusion. As for the latter, it can
be told by tgparentid being nonzero, which is now true both for user-
defined and foreign key's internal triggers.

Author: Amit Langote <amitlangote09(at)gmail(dot)com>
Reviewed-by: Masahiko Sawada <sawada(dot)mshk(at)gmail(dot)com>
Reviewed-by: Arne Roland <A(dot)Roland(at)index(dot)de>
Discussion: https://postgr.es/m/CA+HiwqG7LQSK+n8Bki8tWv7piHD=PnZro2y6ysU2-28JS6cfgQ@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/f4566345cf40b068368cb5617e61318da60676ec

Modified Files
--------------
src/backend/commands/tablecmds.c | 413 +++++++++++++++++++++++++++------
src/backend/commands/trigger.c | 101 +++++++-
src/bin/pg_dump/pg_dump.c | 46 +++-
src/bin/pg_dump/pg_dump.h | 2 +-
src/bin/psql/describe.c | 10 +-
src/include/commands/trigger.h | 4 +
src/test/regress/expected/triggers.out | 4 +-
7 files changed, 490 insertions(+), 90 deletions(-)

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2022-01-05 22:32:17 pgsql: Enable routine running of regex.linux.utf8 regression test.
Previous Message Tom Lane 2022-01-05 18:30:21 pgsql: Enable routine running of citext's UTF8-specific test cases.