diff options
| author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2024-07-24 12:38:18 +0200 |
|---|---|---|
| committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2024-07-24 12:38:18 +0200 |
| commit | 2b22543a440d2fa72a1918950d6e67dc1e71b271 (patch) | |
| tree | 8fcac0e90f46b5264bc0cbc50aab8193ff160897 /src/backend | |
| parent | 20aaa634f72ad9617e34546fafbbda3a3832ee45 (diff) | |
| download | postgresql-2b22543a440d2fa72a1918950d6e67dc1e71b271.tar.gz postgresql-2b22543a440d2fa72a1918950d6e67dc1e71b271.zip | |
Reset relhassubclass upon attaching table as a partition
We don't allow inheritance parents as partitions, and have checks to
prevent this; but if a table _was_ in the past an inheritance parents
and all their children are removed, the pg_class.relhassubclass flag
may remain set, which confuses the partition pruning code (most
obviously, it results in an assertion failure; in production builds it
may be worse.)
Fix by resetting relhassubclass on attach.
Backpatch to all supported versions.
Reported-by: Alexander Lakhin <exclusion@gmail.com>
Reviewed-by: Tom Lane <tgl@sss.pgh.pa.us>
Discussion: https://postgr.es/m/18550-d5e047e9a897a889@postgresql.org
Diffstat (limited to 'src/backend')
| -rw-r--r-- | src/backend/catalog/heap.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/src/backend/catalog/heap.c b/src/backend/catalog/heap.c index ae2efdc760d..ee4b718a9a0 100644 --- a/src/backend/catalog/heap.c +++ b/src/backend/catalog/heap.c @@ -3519,6 +3519,14 @@ StorePartitionBound(Relation rel, Relation parent, PartitionBoundSpec *bound) new_val, new_null, new_repl); /* Also set the flag */ ((Form_pg_class) GETSTRUCT(newtuple))->relispartition = true; + + /* + * We already checked for no inheritance children, but reset + * relhassubclass in case it was left over. + */ + if (rel->rd_rel->relkind == RELKIND_RELATION && rel->rd_rel->relhassubclass) + ((Form_pg_class) GETSTRUCT(newtuple))->relhassubclass = false; + CatalogTupleUpdate(classRel, &newtuple->t_self, newtuple); heap_freetuple(newtuple); table_close(classRel, RowExclusiveLock); |