when creating timeseries_data, never create unnecessary indexes

save space and time during migration process

Author: wiedehopf

acarshub-backend/drizzle/0001_add_timeseries_stats.sql

@@ -16,11 +16,5 @@ CREATE TABLE IF NOT EXISTS `timeseries_stats` (
 	`created_at` integer NOT NULL
 );
 
--- Index for efficient time-range queries
-CREATE INDEX `idx_timeseries_timestamp_resolution` ON `timeseries_stats` (`timestamp`, `resolution`);
-
--- Index for filtering by resolution
-CREATE INDEX `idx_timeseries_resolution` ON `timeseries_stats` (`resolution`);
-
 -- Add constraint check for resolution enum (SQLite doesn't have native ENUM)
 -- Valid values: '1min', '5min', '1hour', '6hour'

acarshub-backend/src/db/migrate.ts

@@ -880,16 +880,6 @@ function migration09_addTimeseriesStats(db: Database.Database): void {
         created_at INTEGER NOT NULL
       )
     `);
-
-    db.exec(`
-      CREATE INDEX idx_timeseries_timestamp_resolution
-      ON timeseries_stats (timestamp, resolution)
-    `);
-
-    db.exec(`
-      CREATE INDEX idx_timeseries_resolution
-      ON timeseries_stats (resolution)
-    `);
   });
 
   migrate();
@@ -1024,26 +1014,7 @@ function migration11_deduplicateTimeseriesAndAddRegistry(
     }
 
     // -------------------------------------------------------------------------
-    // Step 2: Replace non-unique index with unique index
-    // -------------------------------------------------------------------------
-    // Drop the old non-unique index (created by migration 9).  SQLite does not
-    // support ALTER INDEX, so we drop and recreate.
-    db.exec(
-      "DROP INDEX IF EXISTS idx_timeseries_timestamp_resolution",
-    );
-
-    // The table is now duplicate-free, so this will succeed.
-    db.exec(`
-      CREATE UNIQUE INDEX idx_timeseries_timestamp_resolution
-      ON timeseries_stats (timestamp, resolution)
-    `);
-
-    logger.info(
-      "Replaced non-unique index with UNIQUE index on timeseries_stats(timestamp, resolution)",
-    );
-
-    // -------------------------------------------------------------------------
-    // Step 3: Create rrd_import_registry table
+    // Step 2: Create rrd_import_registry table
     // -------------------------------------------------------------------------
     db.exec(`
       CREATE TABLE IF NOT EXISTS rrd_import_registry (
@@ -1142,8 +1113,8 @@ function migration12_dropResolutionPromoteTimestampPk(
 
     // Free space for adsb.im users. Also anyone else.
 
-    db.exec(`DROP INDEX idx_timeseries_resolution;`);
-    db.exec(`DROP INDEX idx_timeseries_timestamp_resolution;`);
+    db.exec(`DROP INDEX IF EXISTS idx_timeseries_resolution;`);
+    db.exec(`DROP INDEX IF EXISTS idx_timeseries_timestamp_resolution;`);
 
     // -----------------------------------------------------------------------
     // Step 2: Create new table — timestamp is INTEGER PRIMARY KEY (rowid alias)

dev-docs/TIMESERIES_STRATEGY.md

@@ -267,12 +267,6 @@ CREATE TABLE timeseries_stats (
   error_count INTEGER DEFAULT 0 NOT NULL,
   created_at INTEGER NOT NULL          -- Insertion timestamp
 );
-
-CREATE INDEX idx_timeseries_timestamp_resolution
-  ON timeseries_stats (timestamp, resolution);
-
-CREATE INDEX idx_timeseries_resolution
-  ON timeseries_stats (resolution);
 ```
 
 ## Configuration