Schema migration of QInfo to eliminate the problematic index

lsst · Aug 1, 2023 · 7344288 · 7344288
1 parent 655206f
commit 7344288
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/qmeta/QMetaMysql.cc b/src/qmeta/QMetaMysql.cc
@@ -47,7 +47,7 @@ using namespace std;
 namespace {
 
 // Current version of QMeta schema
-char const VERSION_STR[] = "8";
+char const VERSION_STR[] = "9";
 
 LOG_LOGGER _log = LOG_GET("lsst.qserv.qmeta.QMetaMysql");
 

diff --git a/src/qmeta/schema/migrate-8-to-9.sql b/src/qmeta/schema/migrate-8-to-9.sql
@@ -0,0 +1,2 @@
+-- This index had significant performance implications for Qserv. 
+DROP INDEX IF EXISTS `QInfo_query_index` ON `QInfo`;
diff --git a/src/qmeta/schema/migrate-None-to-8.sql.jinja → src/qmeta/schema/migrate-None-to-9.sql.jinja b/src/qmeta/schema/migrate-None-to-8.sql.jinja → src/qmeta/schema/migrate-None-to-9.sql.jinja
@@ -59,7 +59,6 @@ CREATE TABLE IF NOT EXISTS `QInfo` (
   KEY `QInfo_completed_index` (`completed`),
   KEY `QInfo_returned_index` (`returned`),
   KEY `QInfo_qtype_index` (`qType`),
-  FULLTEXT KEY `QInfo_query_index` (`query`),
   CONSTRAINT `QInfo_cid`
     FOREIGN KEY (`czarId`)
     REFERENCES `QCzar` (`czarId`)
@@ -219,4 +218,5 @@ COMMENT = 'Table of messages generated during queries.';
 -- Version 6 added indexes to optimize queries made by the Qserv Web Dashboard.
 -- Version 7 added final row count to QInfo.
 -- Version 8 replaced INT with BIGINT in the byte and row counter columns of QInfo.
-INSERT INTO `QMetadata` (`metakey`, `value`) VALUES ('version', '8');
+-- Version 9 removed the full-text index on the query text from QInfo.
+INSERT INTO `QMetadata` (`metakey`, `value`) VALUES ('version', '9');
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		-- This index had significant performance implications for Qserv.
		DROP INDEX IF EXISTS `QInfo_query_index` ON `QInfo`;