Remove redundant indices (#6791)

Author: jdolle

.changeset/nine-timers-complain.md

@@ -0,0 +1,5 @@
+---
+'hive': patch
+---
+
+Remove redundant pg indices

packages/migrations/src/actions/2025.05.15T00-00-00.redundant-indices.ts

@@ -0,0 +1,40 @@
+import { type MigrationExecutor } from '../pg-migrator';
+
+/**
+ * This migration removes indices that are covered by other btree indices.
+ * This should have no impact on performance while lowering our db footprint.
+ */
+export default {
+  name: '2025.05.15T00-00-00.redundant-indices.ts',
+  noTransaction: true,
+  run: ({ sql }) => [
+    // redundant with schema_versions_cursor_pagination
+    {
+      name: 'drop index schema_versions_target_id',
+      query: sql`
+        DROP INDEX CONCURRENTLY IF EXISTS "schema_versions_target_id"
+      `,
+    },
+    // redundant with organization_member_pkey
+    {
+      name: 'drop index organization_member_organization_id',
+      query: sql`
+        DROP INDEX CONCURRENTLY IF EXISTS "organization_member_organization_id"
+      `,
+    },
+    // redundant with schema_checks_connection_pagination
+    {
+      name: 'drop index schema_checks_target_id',
+      query: sql`
+        DROP INDEX CONCURRENTLY IF EXISTS "schema_checks_target_id"
+      `,
+    },
+    // redundant with schema_version_to_log_pkey
+    {
+      name: 'drop index schema_version_to_log_version_id',
+      query: sql`
+        DROP INDEX CONCURRENTLY IF EXISTS "schema_version_to_log_version_id"
+      `,
+    },
+  ],
+} satisfies MigrationExecutor;

packages/migrations/src/run-pg-migrations.ts

@@ -163,5 +163,6 @@ export const runPGMigrations = async (args: { slonik: DatabasePool; runTo?: stri
       await import('./actions/2025.02.21T00-00-00.schema-versions-metadata-attributes'),
       await import('./actions/2025.03.20T00-00-00.dangerous_breaking'),
       await import('./actions/2025.05.14T00-00-00.cascade-deletion-indices-2'),
+      await import('./actions/2025.05.15T00-00-00.redundant-indices'),
     ],
   });