Skip to content

fix: index http_transactions.container_name to speed up pending page #55

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
tito merged 2 commits into from
Jun 2, 2026
Merged

fix: index http_transactions.container_name to speed up pending page #55

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
3abc135
fix: index http_transactions.container_name to speed up pending page
apre Jun 2, 2026
73fddc8
test: update migration count to 15 after adding container index
apre Jun 2, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions internal/greyproxy/crud_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -649,8 +649,8 @@ func TestMigrations(t *testing.T) {
// Verify migration versions were recorded
var count int
_ = db.ReadDB().QueryRow("SELECT COUNT(*) FROM schema_migrations").Scan(&count)
if count != 14 {
t.Errorf("expected 14 migration versions, got %d", count)
if count != 15 {
t.Errorf("expected 15 migration versions, got %d", count)
}
}

Expand Down
6 changes: 6 additions & 0 deletions internal/greyproxy/migrations.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -239,6 +239,12 @@ var migrations = []string{
// returned by the middleware in its hello response. NULL for middlewares
// that did not declare a name; the UI falls back to middleware_url.
`ALTER TABLE middleware_events ADD COLUMN middleware_name TEXT;`,

// Migration 15: Index http_transactions.container_name to speed up the
// container filter on the pending page. Without it, the DISTINCT
// container_name lookup full-scans http_transactions (BLOB-heavy), which
// made GET /pending take ~30s on large databases.
`CREATE INDEX IF NOT EXISTS idx_http_transactions_container ON http_transactions(container_name);`,
}

func runMigrations(db *sql.DB) error {
Expand Down