diff options
| author | Amit Kapila <akapila@postgresql.org> | 2022-08-16 15:25:41 +0530 |
|---|---|---|
| committer | Amit Kapila <akapila@postgresql.org> | 2022-08-16 15:25:41 +0530 |
| commit | 0d5bd3a6cc3e4cd2fb44e2714e52a684cc7bbc25 (patch) | |
| tree | 44ebfaa4ccca6baefb6a30a5f7762b6f0b6baf35 /src/backend/executor/execReplication.c | |
| parent | f4c32cdd1cc3e759d8e300664ec553a16bcb6c37 (diff) | |
| download | postgresql-0d5bd3a6cc3e4cd2fb44e2714e52a684cc7bbc25.tar.gz postgresql-0d5bd3a6cc3e4cd2fb44e2714e52a684cc7bbc25.zip | |
Fix replica identity check for a partitioned table.
The current publisher code checks if UPDATE or DELETE can be executed with
the replica identity of the table even if it's a partitioned table. We can
skip checking the replica identity for partitioned tables because the
operations are actually performed on the leaf partitions (not the
partitioned table).
Reported-by: Brad Nicholson
Author: Hou Zhijie
Reviewed-by: Peter Smith, Amit Kapila
Backpatch-through: 13
Discussion: https://postgr.es/m/CAMMnM%3D8i5DohH%3DYKzV0_wYuYSYvuOJoL9F5nzXTc%2ByzsG1f6rg%40mail.gmail.com
Diffstat (limited to 'src/backend/executor/execReplication.c')
| -rw-r--r-- | src/backend/executor/execReplication.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/src/backend/executor/execReplication.c b/src/backend/executor/execReplication.c index b000645d48b..6014f2e2482 100644 --- a/src/backend/executor/execReplication.c +++ b/src/backend/executor/execReplication.c @@ -569,6 +569,13 @@ CheckCmdReplicaIdentity(Relation rel, CmdType cmd) { PublicationDesc pubdesc; + /* + * Skip checking the replica identity for partitioned tables, because the + * operations are actually performed on the leaf partitions. + */ + if (rel->rd_rel->relkind == RELKIND_PARTITIONED_TABLE) + return; + /* We only need to do checks for UPDATE and DELETE. */ if (cmd != CMD_UPDATE && cmd != CMD_DELETE) return; |