Merge pull request #4513 from alphagov/nhistory-service-index

Add service_id index to notification_history

Author: klssmith

migrations/.current-alembic-head

@@ -1 +1 @@
-0506_n_history_job_id_index
+0507_n_history_service_id_index

migrations/versions/0506_n_history_job_id_index.py

@@ -7,6 +7,7 @@
 revision = '0506_n_history_job_id_index'
 down_revision = '0505_n_history_api_key_index'
 
+# NOTE: This migration was a no-op, since the index already existed
 
 def upgrade():
     with op.get_context().autocommit_block():
@@ -16,7 +17,4 @@ def upgrade():
 
 
 def downgrade():
-    with op.get_context().autocommit_block():
-        op.execute(
-            "DROP INDEX CONCURRENTLY IF EXISTS ix_notification_history_job_id"
-        )
+    pass

migrations/versions/0507_n_history_service_id_index.py

@@ -0,0 +1,22 @@
+"""
+Create Date: 2025-07-07 08:26:08.720911
+"""
+
+from alembic import op
+
+revision = '0507_n_history_service_id_index'
+down_revision = '0506_n_history_job_id_index'
+
+
+def upgrade():
+    with op.get_context().autocommit_block():
+        op.execute(
+            "CREATE INDEX CONCURRENTLY IF NOT EXISTS ix_notification_history_service_id on notification_history (service_id)"
+        )
+
+
+def downgrade():
+    with op.get_context().autocommit_block():
+        op.execute(
+            "DROP INDEX CONCURRENTLY IF EXISTS ix_notification_history_service_id"
+        )