]> granicus.if.org Git - postgresql/commitdiff
Remove repeated test in contrib/amcheck
authorTeodor Sigaev <teodor@sigaev.ru>
Mon, 9 Apr 2018 06:19:09 +0000 (09:19 +0300)
committerTeodor Sigaev <teodor@sigaev.ru>
Mon, 9 Apr 2018 06:19:09 +0000 (09:19 +0300)
Repeating these tests adds unnecessary cycles, since no improvement in
test coverage is expected.

Cleanup from commit 8224de4f42ccf98e08db07b43d52fed72f962ebb.

Peter Geoghegan

contrib/amcheck/expected/check_btree.out
contrib/amcheck/sql/check_btree.sql

index 2a06cce9a0e2ffccb36697e6b17e26f5d2e463da..ed80ac46648aeba43db98b9c0bcd7bc40e9d7c52 100644 (file)
@@ -111,27 +111,9 @@ SELECT bt_index_parent_check('bttest_multi_idx', true);
  
 (1 row)
 
-SELECT bt_index_parent_check('bttest_multi_idx', true);
- bt_index_parent_check 
------------------------
-(1 row)
-
--- repeat same checks with index made by insertions
+-- repeat expansive test for index built using insertions
 TRUNCATE bttest_multi;
 INSERT INTO bttest_multi SELECT i, i%2  FROM generate_series(1, 100000) as i;
-SELECT bt_index_check('bttest_multi_idx');
- bt_index_check 
-----------------
-(1 row)
-
-SELECT bt_index_parent_check('bttest_multi_idx', true);
- bt_index_parent_check 
------------------------
-(1 row)
-
 SELECT bt_index_parent_check('bttest_multi_idx', true);
  bt_index_parent_check 
 -----------------------
index da2f1314e57fc1659e41f5ce3c08c4f6f46169c3..4ca9d2d0ed1f7caad609ec70fff08929bed46f15 100644 (file)
@@ -65,15 +65,11 @@ COMMIT;
 SELECT bt_index_check('bttest_multi_idx');
 -- more expansive test for index with included columns
 SELECT bt_index_parent_check('bttest_multi_idx', true);
-SELECT bt_index_parent_check('bttest_multi_idx', true);
 
--- repeat same checks with index made by insertions
+-- repeat expansive test for index built using insertions
 TRUNCATE bttest_multi;
 INSERT INTO bttest_multi SELECT i, i%2  FROM generate_series(1, 100000) as i;
-SELECT bt_index_check('bttest_multi_idx');
 SELECT bt_index_parent_check('bttest_multi_idx', true);
-SELECT bt_index_parent_check('bttest_multi_idx', true);
-
 
 -- cleanup
 DROP TABLE bttest_a;