Correct error message for row-level triggers with transition tables on partitioned tables.

Etsuro Fujita · Etsuro Fujita · commit 1699125ca218 · 2022-11-04T19:15:03.000+09:00
"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
diff --git a/src/backend/commands/trigger.c b/src/backend/commands/trigger.c
@@ -249,7 +249,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
@@ -2020,7 +2020,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

Original file line number	Diff line number	Diff line change
`@@ -249,7 +249,7 @@ CreateTriggerFiringOn(CreateTrigStmt stmt, const char queryString,`
`249`	`249`	`(errcode(ERRCODE_FEATURE_NOT_SUPPORTED),`
`250`	`250`	`errmsg("\"%s\" is a partitioned table",`
`251`	`251`	`RelationGetRelationName(rel)),`
`252`		`- errdetail("Triggers on partitioned tables cannot have transition tables.")));`
	`252`	`+ errdetail("ROW triggers with transition tables are not supported on partitioned tables.")));`
`253`	`253`	`}`
`254`	`254`	`}`
`255`	`255`	`else if (rel->rd_rel->relkind == RELKIND_VIEW)`