forked from postgres/postgres
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit 29aeda6
committed
Handle arrays and ranges in pg_upgrade's test for non-upgradable types.
pg_upgrade needs to check whether certain non-upgradable data types
appear anywhere on-disk in the source cluster. It knew that it has
to check for these types being contained inside domains and composite
types; but it somehow overlooked that they could be contained in
arrays and ranges, too. Extend the existing recursive-containment
query to handle those cases.
We probably should have noticed this oversight while working on
commit 0ccfc28 and follow-ups, but we failed to :-(. The whole
thing's possibly a bit overdesigned, since we don't really expect
that any of these types will appear on disk; but if we're going to
the effort of doing a recursive search then it's silly not to cover
all the possibilities.
While at it, refactor so that we have only one copy of the search
logic, not three-and-counting. Also, to keep the branches looking
more alike, back-patch the output wording change of commit 1634d36.
Back-patch to all supported branches.
Discussion: https://postgr.es/m/31473.1573412838@sss.pgh.pa.us1 parent 7b8a899 commit 29aeda6Copy full SHA for 29aeda6
File tree
Expand file treeCollapse file tree
1 file changed
+100
-234
lines changedFilter options
- src/bin/pg_upgrade
Expand file treeCollapse file tree
1 file changed
+100
-234
lines changed
0 commit comments