diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2024-03-08 16:32:29 +0100 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2024-03-08 16:32:29 +0100 |
| commit | 270af6f0df764d326e1a7355f4ce10dc73b05dac (patch) | |
| tree | 2ef0efd75bf20275c947c8078ec67ad3d296cd38 /src/backend/replication/logical/relation.c | |
| parent | 4c1973fcaecd9ef11de14ac55d3ec1432f6b82dc (diff) | |
Admit deferrable PKs into rd_pkindex, but flag them as such
... and in particular don't return them as replica identity.
The motivation for this change is letting the primary keys be seen by
code that derives NOT NULL constraints from them, when creating
inheritance children; before this change, if you had a deferrable PK,
pg_dump would not recreate the attnotnull marking properly, because the
column would not be considered as having anything to back said marking
after dropping the throwaway NOT NULL constraint.
The reason we don't want these PKs as replica identities is that
replication can corrupt data, if the uniqueness constraint is
transiently broken.
Reported-by: Amul Sul <sulamul@gmail.com>
Reviewed-by: Dean Rasheed <dean.a.rasheed@gmail.com>
Discussion: https://postgr.es/m/CAAJ_b94QonkgsbDXofakHDnORQNgafd1y3Oa5QXfpQNJyXyQ7A@mail.gmail.com
Diffstat (limited to 'src/backend/replication/logical/relation.c')
| -rw-r--r-- | src/backend/replication/logical/relation.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/replication/logical/relation.c b/src/backend/replication/logical/relation.c index 4f79f46cae2..f139e7b01e9 100644 --- a/src/backend/replication/logical/relation.c +++ b/src/backend/replication/logical/relation.c @@ -843,9 +843,9 @@ IsIndexUsableForReplicaIdentityFull(IndexInfo *indexInfo, AttrMap *attrmap) } /* - * Get replica identity index or if it is not defined a primary key. - * - * If neither is defined, returns InvalidOid + * Return the OID of the replica identity index if one is defined; + * the OID of the PK if one exists and is not deferrable; + * otherwise, InvalidOid. */ Oid GetRelationIdentityOrPK(Relation rel) |