pgsql: Make new GENERATED-expressions code more bulletproof.

Поиск
Список
Период
Сортировка
От Tom Lane
Тема pgsql: Make new GENERATED-expressions code more bulletproof.
Дата
Msg-id E1pH7XD-003pEE-EX@gemulon.postgresql.org
обсуждение исходный текст
Список pgsql-committers
Make new GENERATED-expressions code more bulletproof.

In commit 8bf6ec3ba I assumed that no code path could reach
ExecGetExtraUpdatedCols without having gone through
ExecInitStoredGenerated.  That turns out not to be the case in
logical replication: if there's an ON UPDATE trigger on the target
table, trigger.c will call this code before anybody has set up its
generated columns.  Having seen that, I don't have a lot of faith in
there not being other such paths.  ExecGetExtraUpdatedCols can call
ExecInitStoredGenerated for itself, as long as we are willing to
assume that it is only called in CMD_UPDATE operations, which on
the whole seems like a safer leap of faith.

Per report from Vitaly Davydov.

Discussion: https://postgr.es/m/d259d69652b8c2ff50e14cda3c236c7f@postgrespro.ru

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/3f244d020fbf0b4f01d71e1b7620930be4bf9dc5

Modified Files
--------------
src/backend/executor/execUtils.c         | 13 +++------
src/backend/executor/nodeModifyTable.c   |  2 +-
src/include/executor/nodeModifyTable.h   |  4 +++
src/test/subscription/t/011_generated.pl | 48 +++++++++++++++++++++++++++-----
4 files changed, 50 insertions(+), 17 deletions(-)


В списке pgsql-committers по дате отправления:

Предыдущее
От: Tatsuo Ishii
Дата:
Сообщение: pgsql: Doc: fix typo in backup.sgml.
Следующее
От: Tom Lane
Дата:
Сообщение: pgsql: Make new GENERATED-expressions code more bulletproof.