Remove support for unlogged on partitioned tables
authorMichael Paquier <michael@paquier.xyz>
Thu, 3 Oct 2024 01:55:02 +0000 (10:55 +0900)
committerMichael Paquier <michael@paquier.xyz>
Thu, 3 Oct 2024 01:55:02 +0000 (10:55 +0900)
commite2bab2d792044b55dd092bf1c2be0d570ccb9401
tree70897e26c0eb4149b32ece395e6ec86e28b38504
parent554d3a18f36264eeb1333655b8ddcd929befa6ec
Remove support for unlogged on partitioned tables

The following commands were allowed on partitioned tables, with
different effects:
1) ALTER TABLE SET [UN]LOGGED did not issue an error, and did not update
pg_class.relpersistence.
2) CREATE UNLOGGED TABLE was working with pg_class.relpersistence marked
as initially defined, but partitions did not inherit the UNLOGGED
property, which was confusing.

This commit causes the commands mentioned above to fail for partitioned
tables, instead.

pg_dump is tweaked so as partitioned tables marked as UNLOGGED ignore
the option when dumped from older server versions.  pgbench needs a
tweak for --unlogged and --partitions=N to ignore the UNLOGGED option on
the partitioned tables created, its partitions still being unlogged.

Author: Michael Paquier
Reviewed-by: Nathan Bossart
Discussion: https://postgr.es/m/ZiiyGFTBNkqcMQi_@paquier.xyz
doc/src/sgml/ref/alter_table.sgml
doc/src/sgml/ref/create_table.sgml
src/backend/commands/tablecmds.c
src/bin/pg_dump/pg_dump.c
src/bin/pgbench/pgbench.c
src/test/regress/expected/create_table.out
src/test/regress/sql/create_table.sql