Skip to content

Commit fb583c8

Browse files
committed
Remove abstime, reltime, tinterval tables from old regression databases.
In the back branches, drop these tables after the regression tests are done with them. This fixes failures of cross-branch pg_upgrade testing caused by these types having been removed in v12. We do lose the ability to test dump/restore behavior with these types in the back branches, but the actual loss of code coverage seems to be nil given that there's nothing very special about these types. Discussion: https://postgr.es/m/20181009192237.34wjp3nmw7oynmmr@alap3.anarazel.de
1 parent 01187f3 commit fb583c8

File tree

3 files changed

+17
-4
lines changed

3 files changed

+17
-4
lines changed

src/test/regress/expected/horology.out

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2965,3 +2965,10 @@ SELECT to_char('2012-12-12 12:00'::timestamptz, 'YYYY-MM-DD HH:MI:SS TZ');
29652965
(1 row)
29662966

29672967
RESET TIME ZONE;
2968+
--
2969+
-- Drop tables that we don't want to keep because they interfere with
2970+
-- testing pg_upgrade to v12 and up
2971+
--
2972+
DROP TABLE abstime_tbl;
2973+
DROP TABLE reltime_tbl;
2974+
DROP TABLE tinterval_tbl;

src/test/regress/expected/sanity_check.out

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ SELECT relname, relhasindex
1313
-------------------------+-------------
1414
a | f
1515
a_star | f
16-
abstime_tbl | f
1716
aggtest | f
1817
array_index_op_test | t
1918
array_op_test | f
@@ -142,7 +141,6 @@ SELECT relname, relhasindex
142141
radix_text_tbl | t
143142
ramp | f
144143
real_city | f
145-
reltime_tbl | f
146144
road | t
147145
shighway | t
148146
slow_emp4000 | f
@@ -166,9 +164,8 @@ SELECT relname, relhasindex
166164
timestamp_tbl | f
167165
timestamptz_tbl | f
168166
timetz_tbl | f
169-
tinterval_tbl | f
170167
varchar_tbl | f
171-
(157 rows)
168+
(154 rows)
172169

173170
--
174171
-- another sanity check: every system catalog that has OIDs should have

src/test/regress/sql/horology.sql

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -477,3 +477,12 @@ SELECT '2012-12-12 12:00 America/New_York'::timestamptz;
477477
SELECT to_char('2012-12-12 12:00'::timestamptz, 'YYYY-MM-DD HH:MI:SS TZ');
478478

479479
RESET TIME ZONE;
480+
481+
--
482+
-- Drop tables that we don't want to keep because they interfere with
483+
-- testing pg_upgrade to v12 and up
484+
--
485+
486+
DROP TABLE abstime_tbl;
487+
DROP TABLE reltime_tbl;
488+
DROP TABLE tinterval_tbl;

0 commit comments

Comments
 (0)