diff options
| author | Michael Paquier <michael@paquier.xyz> | 2022-12-23 11:27:18 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2022-12-23 11:27:18 +0900 |
| commit | a2688c4e0329687ca5105b7647bbff6bbe33cca4 (patch) | |
| tree | 00db510b227c6b9e00090f37d8772c66fa678948 | |
| parent | 7445869e1b0d369af9125c9db8196a3be3f44463 (diff) | |
| download | postgresql-a2688c4e0329687ca5105b7647bbff6bbe33cca4.tar.gz postgresql-a2688c4e0329687ca5105b7647bbff6bbe33cca4.zip | |
Fix some incorrectness in upgrade_adapt.sql on query for WITH OIDS
The query used to disable WITH OIDS in all the relations making use of
it was checking for materialized views, but this is not a supported
operation. On the contrary, this needs to be done on foreign tables.
While on it, use quote_ident() in the ALTER TABLE strings built on the
relation name.
Author: Anton A. Melnikov, Michael Paquier
Discussion: https://postgr.es/m/49f389ba-95ce-8a9b-09ae-f60650c0e7c7@inbox.ru
Backpatch-through: 12
| -rw-r--r-- | src/bin/pg_upgrade/upgrade_adapt.sql | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/bin/pg_upgrade/upgrade_adapt.sql b/src/bin/pg_upgrade/upgrade_adapt.sql index 744b4dc9d87..37a4aad2f33 100644 --- a/src/bin/pg_upgrade/upgrade_adapt.sql +++ b/src/bin/pg_upgrade/upgrade_adapt.sql @@ -71,10 +71,10 @@ DO $stmt$ FROM pg_class WHERE relname !~ '^pg_' AND relhasoids - AND relkind in ('r','m') + AND relkind in ('r', 'f') ORDER BY 1 LOOP - execute 'ALTER TABLE ' || rec || ' SET WITHOUT OIDS'; + EXECUTE 'ALTER TABLE ' || quote_ident(rec) || ' SET WITHOUT OIDS'; END LOOP; END; $stmt$; \endif |