pgsql: Fix replica identity check for INSERT ON CONFLICT DO UPDATE.
От | Dean Rasheed |
---|---|
Тема | pgsql: Fix replica identity check for INSERT ON CONFLICT DO UPDATE. |
Дата | |
Msg-id | E1uu7Np-000RtR-11@gemulon.postgresql.org обсуждение исходный текст |
Список | pgsql-committers |
Fix replica identity check for INSERT ON CONFLICT DO UPDATE. If an INSERT has an ON CONFLICT DO UPDATE clause, the executor must check that the target relation supports UPDATE as well as INSERT. In particular, it must check that the target relation has a REPLICA IDENTITY if it publishes updates. Formerly, it was not doing this check, which could lead to silently breaking replication. Fix by adding such a check to CheckValidResultRel(), which requires adding a new onConflictAction argument. In back-branches, preserve ABI compatibility by introducing a wrapper function with the original signature. Author: Zhijie Hou <houzj.fnst@fujitsu.com> Reviewed-by: Ashutosh Bapat <ashutosh.bapat.oss@gmail.com> Reviewed-by: Dean Rasheed <dean.a.rasheed@gmail.com> Tested-by: Chao Li <li.evan.chao@gmail.com> Discussion: https://postgr.es/m/OS3PR01MB57180C87E43A679A730482DF94B62@OS3PR01MB5718.jpnprd01.prod.outlook.com Backpatch-through: 13 Branch ------ REL_18_STABLE Details ------- https://git.postgresql.org/pg/commitdiff/344662848ac15efc89e09b7d39662d7c71b30e06 Modified Files -------------- src/backend/commands/copyfrom.c | 2 +- src/backend/executor/execMain.c | 12 +++++++++++- src/backend/executor/execPartition.c | 9 +++++++-- src/backend/executor/nodeModifyTable.c | 3 ++- src/include/executor/executor.h | 1 + src/test/regress/expected/publication.out | 23 +++++++++++++++++++++++ src/test/regress/sql/publication.sql | 27 +++++++++++++++++++++++++++ 7 files changed, 72 insertions(+), 5 deletions(-)
В списке pgsql-committers по дате отправления: