diff options
| author |  jvoisin <julien.voisin@dustri.org>
| 2024-02-25 16:04:46 +0100 |
|---|---|---|
| committer |  Frédéric Guillot <f@miniflux.net>
| 2024-02-25 17:50:30 -0800 |
| commit | 04916a57d28b75a5060358cd0b5d898ee55427cb (patch) | |
| tree | d604bae6f47813c6ca354fb218b770c77f712dc1 | |
| parent | 0adac5c6f7d3c03298a73a223d627813cc7ae0d6 (diff) | |
| download | v2-04916a57d28b75a5060358cd0b5d898ee55427cb.tar.gz v2-04916a57d28b75a5060358cd0b5d898ee55427cb.tar.zst v2-04916a57d28b75a5060358cd0b5d898ee55427cb.zip | |
Simplify CleanOldUserSessions' query
No need for a subquery, filtering on `created_at` directly is enough.
| -rw-r--r-- | internal/storage/session.go | 2 | ||||
| -rw-r--r-- | 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 { |