From 145dc515a2353b4ef11efbee00f284fc5326f0bc Mon Sep 17 00:00:00 2001
From: Sam Lade <sam@sentynel.com>
Date: Tue, 31 Oct 2023 13:43:33 +0000
Subject: [PATCH] Remove account_suspended_at_idx to resolve slow query issues
 (#2310)

---
 ...0231029143819_remove_suspended_at_index.go | 50 +++++++++++++++++++
 1 file changed, 50 insertions(+)
 create mode 100644 internal/db/bundb/migrations/20231029143819_remove_suspended_at_index.go

diff --git a/internal/db/bundb/migrations/20231029143819_remove_suspended_at_index.go b/internal/db/bundb/migrations/20231029143819_remove_suspended_at_index.go
new file mode 100644
index 000000000..9555e507d
--- /dev/null
+++ b/internal/db/bundb/migrations/20231029143819_remove_suspended_at_index.go
@@ -0,0 +1,50 @@
+// GoToSocial
+// Copyright (C) GoToSocial Authors admin@gotosocial.org
+// SPDX-License-Identifier: AGPL-3.0-or-later
+//
+// This program is free software: you can redistribute it and/or modify
+// it under the terms of the GNU Affero General Public License as published by
+// the Free Software Foundation, either version 3 of the License, or
+// (at your option) any later version.
+//
+// This program is distributed in the hope that it will be useful,
+// but WITHOUT ANY WARRANTY; without even the implied warranty of
+// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+// GNU Affero General Public License for more details.
+//
+// You should have received a copy of the GNU Affero General Public License
+// along with this program.  If not, see <http://www.gnu.org/licenses/>.
+
+package migrations
+
+import (
+	"context"
+
+	"github.com/uptrace/bun"
+)
+
+func init() {
+	up := func(ctx context.Context, db *bun.DB) error {
+		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
+			// Remove unhelpful index which can confuse the query optimiser
+			// for CountInstanceUsers()
+			if _, err := tx.
+				NewDropIndex().
+				Index("accounts_suspended_at_idx").
+				Exec(ctx); err != nil {
+				return err
+			}
+			return nil
+		})
+	}
+
+	down := func(ctx context.Context, db *bun.DB) error {
+		return db.RunInTx(ctx, nil, func(ctx context.Context, tx bun.Tx) error {
+			return nil
+		})
+	}
+
+	if err := Migrations.Register(up, down); err != nil {
+		panic(err)
+	}
+}