From f8c414b2c92ebbba7be41711f9d03507ebe8dfd1 Mon Sep 17 00:00:00 2001 From: David Dworken Date: Mon, 20 Jan 2025 18:50:33 -0700 Subject: [PATCH] Skip deep cleans for sqlite DBs (#289) * Add explicit DROP TABLE directives for temp_inactive_devices and drop the TEMP qualifier for non-postgres DBs, fixes #288 * Revert "Add explicit DROP TABLE directives for temp_inactive_devices and drop the TEMP qualifier for non-postgres DBs, fixes #288" This reverts commit db6d8413a74d1fe624e431243bb0808bee8ab01f. * Skip deep cleans for sqlite DBs --- backend/server/internal/database/db.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/backend/server/internal/database/db.go b/backend/server/internal/database/db.go index 096b49d..8988747 100644 --- a/backend/server/internal/database/db.go +++ b/backend/server/internal/database/db.go @@ -429,6 +429,10 @@ func (db *DB) GenerateAndStoreActiveUserStats(ctx context.Context) error { } func (db *DB) SelfHostedDeepClean(ctx context.Context) error { + if db.Name() == "sqlite" { + // sqlite doesn't support the `(now() - INTERVAL '90 days')` syntax used in the below queries. + return nil + } return db.WithContext(ctx).Transaction(func(tx *gorm.DB) error { runDeletes := os.Getenv("HISHTORY_SELF_HOSTED_DEEP_CLEAN") != "" r := tx.Exec(`