From 04916a57d28b75a5060358cd0b5d898ee55427cb Mon Sep 17 00:00:00 2001 From: jvoisin Date: Sun, 25 Feb 2024 16:04:46 +0100 Subject: [PATCH] Simplify CleanOldUserSessions' query No need for a subquery, filtering on `created_at` directly is enough. --- internal/storage/session.go | 2 +- internal/storage/user_session.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/storage/session.go b/internal/storage/session.go index cf48b5aa..1144f3ff 100644 --- a/internal/storage/session.go +++ b/internal/storage/session.go @@ -128,7 +128,7 @@ func (s *Storage) CleanOldSessions(days int) int64 { DELETE FROM sessions WHERE - id IN (SELECT id FROM sessions WHERE created_at < now() - interval '%d days') + created_at < now() - interval '%d days' ` result, err := s.db.Exec(fmt.Sprintf(query, days)) if err != nil { diff --git a/internal/storage/user_session.go b/internal/storage/user_session.go index 9853d758..c0339873 100644 --- a/internal/storage/user_session.go +++ b/internal/storage/user_session.go @@ -170,7 +170,7 @@ func (s *Storage) CleanOldUserSessions(days int) int64 { DELETE FROM user_sessions WHERE - id IN (SELECT id FROM user_sessions WHERE created_at < now() - interval '%d days') + created_at < now() - interval '%d days' ` result, err := s.db.Exec(fmt.Sprintf(query, days)) if err != nil {