Fix sqlite bootstrap

efixler · May 6, 2024 · 6a4b557 · 6a4b557
1 parent bb1ba9e
commit 6a4b557
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/internal/storage/sqlite/migrations/20240505192733_init.sql b/internal/storage/sqlite/migrations/20240505192733_init.sql
@@ -1,10 +1,5 @@
 -- +goose Up
 -- +goose StatementBegin
-PRAGMA foreign_keys = off;
-PRAGMA page_size = 16384; -- default is 4096 but text records are big
-PRAGMA temp_store = MEMORY;
-PRAGMA auto_vacuum = INCREMENTAL;
-
 CREATE TABLE IF NOT EXISTS id_map (
     requested_id INTEGER PRIMARY KEY ON CONFLICT REPLACE
                          NOT NULL,

diff --git a/internal/storage/sqlite/storage.go b/internal/storage/sqlite/storage.go
@@ -68,9 +68,13 @@ func (s *Store) Open(ctx context.Context) error {
 	// In Memory DBs must always be created
 	needsCreate := s.config.IsInMemory() || !exists(s.config.filename)
 	if needsCreate {
+		// Todo: Merge create and migrate into a single operation
 		if err := s.Create(); err != nil {
 			return err
 		}
+		if err := s.Migrate(); err != nil {
+			return err
+		}
 	}
 	s.Maintenance(24*time.Hour, maintain)
 	return nil