Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit 8117326

Browse files
author
Etsuro Fujita
committed
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 86f5759; back-patch to v11 where that commit came in. Reviewed by Tom Lane. Discussion: https://postgr.es/m/CAPmGK17gk4vXLzz2iG%2BG4LWRWCoVyam70nZ3OuGm1hMJwDrhcg%40mail.gmail.com
1 parent 387e059 commit 8117326

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/backend/commands/trigger.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ CreateTriggerFiringOn(CreateTrigStmt *stmt, const char *queryString,
262262
(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),
263263
errmsg("\"%s\" is a partitioned table",
264264
RelationGetRelationName(rel)),
265-
errdetail("Triggers on partitioned tables cannot have transition tables.")));
265+
errdetail("ROW triggers with transition tables are not supported on partitioned tables.")));
266266
}
267267
}
268268
else if (rel->rd_rel->relkind == RELKIND_VIEW)

src/test/regress/expected/triggers.out

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2046,7 +2046,7 @@ create trigger failed after update on parted_trig
20462046
referencing old table as old_table
20472047
for each row execute procedure trigger_nothing();
20482048
ERROR: "parted_trig" is a partitioned table
2049-
DETAIL: Triggers on partitioned tables cannot have transition tables.
2049+
DETAIL: ROW triggers with transition tables are not supported on partitioned tables.
20502050
drop table parted_trig;
20512051
--
20522052
-- Verify trigger creation for partitioned tables, and drop behavior

0 commit comments

Comments
 (0)