Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Commit e84c71b

Browse files
committed
Update oidjoins regression test to match current catalog structure.
1 parent bb892ce commit e84c71b

File tree

3 files changed

+32
-4
lines changed

3 files changed

+32
-4
lines changed

contrib/findoidjoins/README.findoidjoins

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,10 +7,10 @@ anything but an empty database, such as template1; else it's likely to
77
be very slow.
88

99
Run on an empty database, it returns the system join relationships (shown
10-
below for 7.4). Note that unexpected matches may indicate bogus entries
10+
below for 8.0). Note that unexpected matches may indicate bogus entries
1111
in system tables --- don't accept a peculiar match without question.
1212
In particular, a field shown as joining to more than one target table is
13-
probably messed up. In 7.4, the *only* fields that should join to more
13+
probably messed up. In 8.0, the *only* fields that should join to more
1414
than one target are pg_description.objoid, pg_depend.objid, and
1515
pg_depend.refobjid. (Running make_oidjoins_check is an easy way to spot
1616
fields joining to more than one table, BTW.)
@@ -26,11 +26,11 @@ revision in the patterns of cross-links between system tables.
2626
(Ideally we'd just regenerate the script as part of the regression
2727
tests themselves, but that seems too slow...)
2828

29-
NOTE: in 7.4, make_oidjoins_check produces one bogus join check:
29+
NOTE: in 8.0, make_oidjoins_check produces one bogus join check:
3030
Join pg_catalog.pg_class.relfilenode => pg_catalog.pg_class.oid
3131
This is an artifact and should not be added to the oidjoins regress test.
3232
Also beware of any claim that pg_database.datlastsysoid joins to anything;
33-
this does not actually happen in 7.4, but it did happen before and might
33+
this does not actually happen in 8.0, but it did happen before and might
3434
happen again in future, depending on what operation initdb does last.
3535

3636
---------------------------------------------------------------------------
@@ -51,8 +51,10 @@ Join pg_catalog.pg_am.ambulkdelete => pg_catalog.pg_proc.oid
5151
Join pg_catalog.pg_am.amvacuumcleanup => pg_catalog.pg_proc.oid
5252
Join pg_catalog.pg_am.amcostestimate => pg_catalog.pg_proc.oid
5353
Join pg_catalog.pg_amop.amopclaid => pg_catalog.pg_opclass.oid
54+
Join pg_catalog.pg_amop.amopsubtype => pg_catalog.pg_type.oid
5455
Join pg_catalog.pg_amop.amopopr => pg_catalog.pg_operator.oid
5556
Join pg_catalog.pg_amproc.amopclaid => pg_catalog.pg_opclass.oid
57+
Join pg_catalog.pg_amproc.amprocsubtype => pg_catalog.pg_type.oid
5658
Join pg_catalog.pg_amproc.amproc => pg_catalog.pg_proc.oid
5759
Join pg_catalog.pg_attribute.attrelid => pg_catalog.pg_class.oid
5860
Join pg_catalog.pg_attribute.atttypid => pg_catalog.pg_type.oid
@@ -62,12 +64,14 @@ Join pg_catalog.pg_cast.castfunc => pg_catalog.pg_proc.oid
6264
Join pg_catalog.pg_class.relnamespace => pg_catalog.pg_namespace.oid
6365
Join pg_catalog.pg_class.reltype => pg_catalog.pg_type.oid
6466
Join pg_catalog.pg_class.relam => pg_catalog.pg_am.oid
67+
Join pg_catalog.pg_class.reltablespace => pg_catalog.pg_tablespace.oid
6568
Join pg_catalog.pg_class.reltoastrelid => pg_catalog.pg_class.oid
6669
Join pg_catalog.pg_class.reltoastidxid => pg_catalog.pg_class.oid
6770
Join pg_catalog.pg_constraint.connamespace => pg_catalog.pg_namespace.oid
6871
Join pg_catalog.pg_constraint.contypid => pg_catalog.pg_type.oid
6972
Join pg_catalog.pg_conversion.connamespace => pg_catalog.pg_namespace.oid
7073
Join pg_catalog.pg_conversion.conproc => pg_catalog.pg_proc.oid
74+
Join pg_catalog.pg_database.dattablespace => pg_catalog.pg_tablespace.oid
7175
Join pg_catalog.pg_depend.classid => pg_catalog.pg_class.oid
7276
Join pg_catalog.pg_depend.refclassid => pg_catalog.pg_class.oid
7377
Join pg_catalog.pg_description.classoid => pg_catalog.pg_class.oid

src/test/regress/expected/oidjoins.out

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -233,6 +233,14 @@ WHERE relam != 0 AND
233233
------+-------
234234
(0 rows)
235235

236+
SELECT ctid, reltablespace
237+
FROM pg_catalog.pg_class fk
238+
WHERE reltablespace != 0 AND
239+
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_tablespace pk WHERE pk.oid = fk.reltablespace);
240+
ctid | reltablespace
241+
------+---------------
242+
(0 rows)
243+
236244
SELECT ctid, reltoastrelid
237245
FROM pg_catalog.pg_class fk
238246
WHERE reltoastrelid != 0 AND
@@ -281,6 +289,14 @@ WHERE conproc != 0 AND
281289
------+---------
282290
(0 rows)
283291

292+
SELECT ctid, dattablespace
293+
FROM pg_catalog.pg_database fk
294+
WHERE dattablespace != 0 AND
295+
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_tablespace pk WHERE pk.oid = fk.dattablespace);
296+
ctid | dattablespace
297+
------+---------------
298+
(0 rows)
299+
284300
SELECT ctid, classid
285301
FROM pg_catalog.pg_depend fk
286302
WHERE classid != 0 AND

src/test/regress/sql/oidjoins.sql

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,10 @@ SELECT ctid, relam
117117
FROM pg_catalog.pg_class fk
118118
WHERE relam != 0 AND
119119
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_am pk WHERE pk.oid = fk.relam);
120+
SELECT ctid, reltablespace
121+
FROM pg_catalog.pg_class fk
122+
WHERE reltablespace != 0 AND
123+
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_tablespace pk WHERE pk.oid = fk.reltablespace);
120124
SELECT ctid, reltoastrelid
121125
FROM pg_catalog.pg_class fk
122126
WHERE reltoastrelid != 0 AND
@@ -141,6 +145,10 @@ SELECT ctid, conproc
141145
FROM pg_catalog.pg_conversion fk
142146
WHERE conproc != 0 AND
143147
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_proc pk WHERE pk.oid = fk.conproc);
148+
SELECT ctid, dattablespace
149+
FROM pg_catalog.pg_database fk
150+
WHERE dattablespace != 0 AND
151+
NOT EXISTS(SELECT 1 FROM pg_catalog.pg_tablespace pk WHERE pk.oid = fk.dattablespace);
144152
SELECT ctid, classid
145153
FROM pg_catalog.pg_depend fk
146154
WHERE classid != 0 AND

0 commit comments

Comments
 (0)