Add comments warning against generalizing default_with_oids.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 26 Apr 2015 01:37:39 +0000 (21:37 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 26 Apr 2015 01:38:06 +0000 (21:38 -0400)
commit0bd11d9711b88e72d2022e25b9227c480aca4978
tree762e3fdbdd38a313d68c41ac7ecca46fe4f8892d
parent04f1542d390fdd95dadc66e86c623ecdc1bca401
Add comments warning against generalizing default_with_oids.

pg_dump has historically assumed that default_with_oids affects only plain
tables and not other relkinds.  Conceivably we could make it apply to some
newly invented relkind if we did so from the get-go, but changing the
behavior for existing object types will break existing dump scripts.
Add code comments warning about this interaction.

Also, make sure that default_with_oids doesn't cause parse_utilcmd.c to
think that CREATE FOREIGN TABLE will create an OID column.  I think this is
only a latent bug right now, since we don't allow UNIQUE/PKEY constraints
in CREATE FOREIGN TABLE, but it's better to be consistent and future-proof.
src/backend/commands/tablecmds.c
src/backend/parser/parse_utilcmd.c