Skip to content

Commit 85b3bcb

Browse files
committed
Remove useless and rather expensive stanza in matview regression test.
This removes a test case added by commit b69ec7c, which was intended to exercise a corner case involving the rule used at that time that materialized views were unpopulated iff they had physical size zero. We got rid of that rule very shortly later, in commit 1d6c72a, but kept the test case. However, because the case now asks what VACUUM will do to a zero-sized physical file, it would be pretty surprising if the answer were ever anything but "nothing" ... and if things were indeed that broken, surely we'd find it out from other tests. Since the test involves a table that's fairly large by regression-test standards (100K rows), it's quite slow to run. Dropping it should save some buildfarm cycles, so let's do that. Discussion: https://postgr.es/m/32386.1493831320@sss.pgh.pa.us
1 parent 35ac926 commit 85b3bcb

File tree

2 files changed

+0
-32
lines changed

2 files changed

+0
-32
lines changed

src/test/regress/expected/matview.out

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -365,26 +365,6 @@ NOTICE: drop cascades to 3 other objects
365365
DETAIL: drop cascades to view v_test2
366366
drop cascades to materialized view mv_test2
367367
drop cascades to materialized view mv_test3
368-
-- test that vacuum does not make empty matview look unpopulated
369-
CREATE TABLE hoge (i int);
370-
INSERT INTO hoge VALUES (generate_series(1,100000));
371-
CREATE MATERIALIZED VIEW hogeview AS SELECT * FROM hoge WHERE i % 2 = 0;
372-
CREATE INDEX hogeviewidx ON hogeview (i);
373-
DELETE FROM hoge;
374-
REFRESH MATERIALIZED VIEW hogeview;
375-
SELECT * FROM hogeview WHERE i < 10;
376-
i
377-
---
378-
(0 rows)
379-
380-
VACUUM ANALYZE hogeview;
381-
SELECT * FROM hogeview WHERE i < 10;
382-
i
383-
---
384-
(0 rows)
385-
386-
DROP TABLE hoge CASCADE;
387-
NOTICE: drop cascades to materialized view hogeview
388368
-- allow subquery to reference unpopulated matview if WITH NO DATA is specified
389369
CREATE MATERIALIZED VIEW mv1 AS SELECT 1 AS col1 WITH NO DATA;
390370
CREATE MATERIALIZED VIEW mv2 AS SELECT * FROM mv1

src/test/regress/sql/matview.sql

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -113,18 +113,6 @@ SELECT relispopulated FROM pg_class WHERE oid = 'mv_test3'::regclass;
113113

114114
DROP VIEW v_test1 CASCADE;
115115

116-
-- test that vacuum does not make empty matview look unpopulated
117-
CREATE TABLE hoge (i int);
118-
INSERT INTO hoge VALUES (generate_series(1,100000));
119-
CREATE MATERIALIZED VIEW hogeview AS SELECT * FROM hoge WHERE i % 2 = 0;
120-
CREATE INDEX hogeviewidx ON hogeview (i);
121-
DELETE FROM hoge;
122-
REFRESH MATERIALIZED VIEW hogeview;
123-
SELECT * FROM hogeview WHERE i < 10;
124-
VACUUM ANALYZE hogeview;
125-
SELECT * FROM hogeview WHERE i < 10;
126-
DROP TABLE hoge CASCADE;
127-
128116
-- allow subquery to reference unpopulated matview if WITH NO DATA is specified
129117
CREATE MATERIALIZED VIEW mv1 AS SELECT 1 AS col1 WITH NO DATA;
130118
CREATE MATERIALIZED VIEW mv2 AS SELECT * FROM mv1

0 commit comments

Comments
 (0)