postgres.git / summary / log / commit / refs

commit    572695bcbcbd887fbf45653138e960831dd71db1
Author:   Etsuro Fujita <efujita@postgresql.org>
Date:     Fri Nov 04 10:15:06 2022 +0000
First released in: 16.0, 11.18, 12.13, 13.9, 14.6, 15.1

    Correct error message for row-level triggers with transition tables on partitioned tables.
    
    "Triggers on partitioned tables cannot have transition tables." is
    incorrect as we allow statement-level triggers on partitioned tables to
    have transition tables.
    
    This has been wrong since commit 86f575948; back-patch to v11 where that
    commit came in.
    
    Reviewed by Tom Lane.
    
    Discussion: https://postgr.es/m/CAPmGK17gk4vXLzz2iG%2BG4LWRWCoVyam70nZ3OuGm1hMJwDrhcg%40mail.gmail.com


src/backend/commands/trigger.c | 2 +- src/test/regress/expected/triggers.out | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c index 1b72984ca37..794bfe1f149 100644 --- a/src/backend/commands/trigger.c +++ b/src/backend/commands/trigger.c @@ -263,7 +263,7 @@ CreateTriggerFiringOn(CreateTrigStmt *stmt, const char *queryString, (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), errmsg("\"%s\" is a partitioned table", RelationGetRelationName(rel)), - errdetail("Triggers on partitioned tables cannot have transition tables."))); + errdetail("ROW triggers with transition tables are not supported on partitioned tables."))); } } else if (rel->rd_rel->relkind == RELKIND_VIEW) diff --git a/src/test/regress/expected/triggers.out b/src/test/regress/expected/triggers.out index be6132db696..79e5e4b3533 100644 --- a/src/test/regress/expected/triggers.out +++ b/src/test/regress/expected/triggers.out @@ -2024,7 +2024,7 @@ create trigger failed after update on parted_trig referencing old table as old_table for each row execute procedure trigger_nothing(); ERROR: "parted_trig" is a partitioned table -DETAIL: Triggers on partitioned tables cannot have transition tables. +DETAIL: ROW triggers with transition tables are not supported on partitioned tables. drop table parted_trig; -- -- Verify trigger creation for partitioned tables, and drop behavior [parent: ab70b3a52fde]