pgsql: Fix ENABLE/DISABLE TRIGGER to handle recursion correctly
От | Alvaro Herrera |
---|---|
Тема | pgsql: Fix ENABLE/DISABLE TRIGGER to handle recursion correctly |
Дата | |
Msg-id | E1oJs4Y-000OdC-Pj@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_12_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/6e7b37264e96d3216240b207f763913164cf6487 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 по дате отправления: