Skip to content

Commit 114fb1c

Browse files
author
Mark Saroufim
committed
Remove duplicate indexes from migration
idx_runs_submission_id and idx_runs_ranking already existed on prod (as idx_runs_submission_id_covering and idx_runs_leaderboard_valid). Keep only the two genuinely new indexes.
1 parent 9f28b1c commit 114fb1c

1 file changed

Lines changed: 0 additions & 15 deletions

File tree

src/migrations/20260225_01_aW5Bu-add-leaderboard-performance-indexes.py

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,6 @@
77
__depends__ = {"20260108_01_gzSm3-add-submission-status"}
88

99
steps = [
10-
# FK index on runs.submission_id
11-
# Used in every ranking query (JOIN runs ON submission_id = s.id)
12-
step(
13-
"CREATE INDEX IF NOT EXISTS idx_runs_submission_id ON leaderboard.runs(submission_id)",
14-
"DROP INDEX IF EXISTS leaderboard.idx_runs_submission_id",
15-
),
1610
# FK index on submission.leaderboard_id
1711
# Every query filters WHERE leaderboard_id = X
1812
step(
@@ -25,13 +19,4 @@
2519
"CREATE INDEX IF NOT EXISTS idx_submission_user_id ON leaderboard.submission(user_id)",
2620
"DROP INDEX IF EXISTS leaderboard.idx_submission_user_id",
2721
),
28-
# Partial composite index for ranking queries
29-
# Covers WHERE NOT r.secret AND r.score IS NOT NULL AND r.passed
30-
# Includes (submission_id, runner, score) for JOIN, PARTITION BY, ORDER BY
31-
step(
32-
"CREATE INDEX IF NOT EXISTS idx_runs_ranking"
33-
" ON leaderboard.runs(submission_id, runner, score)"
34-
" WHERE NOT secret AND score IS NOT NULL AND passed",
35-
"DROP INDEX IF EXISTS leaderboard.idx_runs_ranking",
36-
),
3722
]

0 commit comments

Comments
 (0)