pgsql: Fix ENABLE/DISABLE TRIGGER to handle recursion correctly
От | Alvaro Herrera |
---|---|
Тема | pgsql: Fix ENABLE/DISABLE TRIGGER to handle recursion correctly |
Дата | |
Msg-id | E1oJs4Y-000Od5-O5@gemulon.postgresql.org обсуждение исходный текст |
Список | pgsql-committers |
Fix ENABLE/DISABLE TRIGGER to handle recursion correctly Using ATSimpleRecursion() in ATPrepCmd() to do so as bbb927b4db9b did is not correct, because ATPrepCmd() can't distinguish between triggers that may be cloned and those that may not, so would wrongly try to recurse for the latter category of triggers. So this commit restores the code in EnableDisableTrigger() that 86f575948c77 had added to do the recursion, which would do it only for triggers that may be cloned, that is, row-level triggers. This also changes tablecmds.c such that ATExecCmd() is able to pass the value of ONLY flag down to EnableDisableTrigger() using its new 'recurse' parameter. This also fixes what seems like an oversight of 86f575948c77 that the recursion to partition triggers would only occur if EnableDisableTrigger() had actually changed the trigger. It is more apt to recurse to inspect partition triggers even if the parent's trigger didn't need to be changed: only then can we be certain that all descendants share the same state afterwards. Backpatch all the way back to 11, like bbb927b4db9b. Care is taken not to break ABI compatibility (and that no catversion bump is needed.) Co-authored-by: Amit Langote <amitlangote09@gmail.com> Reviewed-by: Dmitry Koval <d.koval@postgrespro.ru> Discussion: https://postgr.es/m/CA+HiwqG-cZT3XzGAnEgZQLoQbyfJApVwOTQaCaas1mhpf+4V5A@mail.gmail.com Branch ------ REL_15_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/e78fd9084587cd485fdec3eb3eaeef567e1707d5 Modified Files -------------- src/backend/commands/tablecmds.c | 64 +++++++++++++++++++++++----------- src/backend/commands/trigger.c | 47 +++++++++++++++++++++++-- src/backend/nodes/copyfuncs.c | 1 + src/backend/nodes/equalfuncs.c | 1 + src/include/commands/trigger.h | 3 ++ src/include/nodes/parsenodes.h | 1 + src/test/regress/expected/triggers.out | 40 +++++++++++++++------ src/test/regress/sql/triggers.sql | 11 +++++- 8 files changed, 134 insertions(+), 34 deletions(-)
В списке pgsql-committers по дате отправления: