Don't set a fast default for anything but a plain table
authorAndrew Dunstan <andrew@dunslane.net>
Fri, 18 Jun 2021 10:51:12 +0000 (06:51 -0400)
committerAndrew Dunstan <andrew@dunslane.net>
Fri, 18 Jun 2021 10:51:12 +0000 (06:51 -0400)
commit0a4efdc7ebf2584257b166c87e82797eb92815b5
tree72ba741ef759f1ccb2709bc4ac0e4357974ea289
parent981524d2e3aa3f28d364c472e34f5386be846811
Don't set a fast default for anything but a plain table

The fast default code added in Release 11 omitted to check that the
table a fast default was being added to was a plain table. Thus one
could be added to a foreign table, which predicably blows up. Here we
perform that check.

In addition, on the back branches, since some of these might have
escaped into the wild, if we encounter a missing value for
an attribute of something other than a plain table we ignore it.

Fixes bug #17056

Backpatch to release 11,

Reviewed by: Andres Freund, Álvaro Herrera and Tom Lane
src/backend/catalog/heap.c
src/test/regress/expected/fast_default.out
src/test/regress/sql/fast_default.sql