LP#1479953: Rename indexes to *_idx and add pgTAP test.
authorJason Stephenson <jstephenson@mvlc.org>
Thu, 6 Aug 2015 15:13:00 +0000 (11:13 -0400)
committerJason Stephenson <jstephenson@mvlc.org>
Thu, 6 Aug 2015 15:44:26 +0000 (11:44 -0400)
To make the new index names more consistent with existing indices,
I took the liberty of adding _idx to the new index names.

We also add a pgTAP test to verify that the new indices exist in
the database.  The test would fail before running the upgrade
script and succeed after.

Signed-off-by: Jason Stephenson <jstephenson@mvlc.org>
Open-ILS/src/sql/Pg/012.schema.vandelay.sql
Open-ILS/src/sql/Pg/200.schema.acq.sql
Open-ILS/src/sql/Pg/201.acq.audit-functions.sql
Open-ILS/src/sql/Pg/t/regress/lp1479953_has_indexes.pg [new file with mode: 0644]
Open-ILS/src/sql/Pg/upgrade/XXXX.index.queued_record.sql

index 25e1a6d..27eb528 100644 (file)
@@ -150,7 +150,7 @@ CREATE TABLE vandelay.bib_match (
     quality         INT         NOT NULL DEFAULT 1,
     match_score     INT         NOT NULL DEFAULT 0
 );
-CREATE INDEX bib_match_queued_record ON vandelay.bib_match (queued_record);
+CREATE INDEX bib_match_queued_record_idx ON vandelay.bib_match (queued_record);
 
 CREATE TABLE vandelay.import_item (
     id              BIGSERIAL   PRIMARY KEY,
@@ -181,7 +181,7 @@ CREATE TABLE vandelay.import_item (
     opac_visible    BOOL,
     internal_id     BIGINT -- queue_type == 'acq' ? acq.lineitem_detail.id : asset.copy.id
 );
-CREATE INDEX import_item_record ON vandelay.import_item (record);
+CREATE INDEX import_item_record_idx ON vandelay.import_item (record);
 
 CREATE TABLE vandelay.import_bib_trash_group(
     id           SERIAL  PRIMARY KEY,
index 9c9f9e5..2741ebc 100644 (file)
@@ -489,7 +489,7 @@ CREATE INDEX li_pl_idx ON acq.lineitem (picklist);
 CREATE INDEX li_creator_idx   ON acq.lineitem ( creator );
 CREATE INDEX li_editor_idx    ON acq.lineitem ( editor );
 CREATE INDEX li_selector_idx  ON acq.lineitem ( selector );
-CREATE INDEX li_queued_record ON acq.lineitem ( queued_record );
+CREATE INDEX li_queued_record_idx ON acq.lineitem ( queued_record );
 
 CREATE TABLE acq.lineitem_alert_text (
     id               SERIAL         PRIMARY KEY,
index 383e29f..c9d17b6 100644 (file)
@@ -103,6 +103,6 @@ CREATE INDEX acq_po_hist_id_idx            ON acq.acq_purchase_order_history( id
 
 SELECT acq.create_acq_auditor ( 'acq', 'lineitem' );
 CREATE INDEX acq_lineitem_hist_id_idx            ON acq.acq_lineitem_history( id );
-CREATE INDEX acq_lineitem_history_queued_record  ON acq.acq_lineitem_history (queued_record);
+CREATE INDEX acq_lineitem_history_queued_record_idx  ON acq.acq_lineitem_history (queued_record);
 
 COMMIT;
diff --git a/Open-ILS/src/sql/Pg/t/regress/lp1479953_has_indexes.pg b/Open-ILS/src/sql/Pg/t/regress/lp1479953_has_indexes.pg
new file mode 100644 (file)
index 0000000..0921d61
--- /dev/null
@@ -0,0 +1,10 @@
+BEGIN;
+
+SELECT plan(4);
+
+SELECT has_index('acq', 'acq_lineitem_history', 'acq_lineitem_history_queued_record_idx', ARRAY['queued_record']);
+SELECT has_index('acq', 'lineitem', 'li_queued_record_idx', ARRAY['queued_record']);
+SELECT has_index('vandelay', 'bib_match', 'bib_match_queued_record_idx', ARRAY['queued_record']);
+SELECT has_index('vandelay', 'import_item', 'import_item_record_idx', ARRAY['record']);
+
+ROLLBACK;
index 4ca13c7..a1fb2c2 100644 (file)
@@ -7,9 +7,9 @@ BEGIN;
 
 SELECT evergreen.upgrade_deps_block_check('XXXX', :eg_version);
 
-CREATE INDEX acq_lineitem_history_queued_record ON acq.acq_lineitem_history (queued_record);
-CREATE INDEX li_queued_record ON acq.lineitem (queued_record);
-CREATE INDEX bib_match_queued_record ON vandelay.bib_match (queued_record);
-CREATE INDEX import_item_record ON vandelay.import_item (record);
+CREATE INDEX acq_lineitem_history_queued_record_idx ON acq.acq_lineitem_history (queued_record);
+CREATE INDEX li_queued_record_idx ON acq.lineitem (queued_record);
+CREATE INDEX bib_match_queued_record_idx ON vandelay.bib_match (queued_record);
+CREATE INDEX import_item_record_idx ON vandelay.import_item (record);
 
 COMMIT;