johno/beads
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add SQLite native time format to connection string (#61)

Browse Source 
Use _time_format=sqlite parameter in modernc.org/sqlite connection
string to ensure DATETIME columns use SQLite's native time format
(format 7 with timezone) instead of Go's default String() format.

This improves compatibility with SQLite's date/time functions and
ensures consistent time representation across the database.
This commit is contained in:
Juan Vargas
2025-10-17 15:35:43 -03:00
committed by GitHub
parent 789145f842
commit e6a69401c9
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
internal/storage/sqlite/sqlite.go
View File

@@ -32,7 +32,8 @@ func New(path string) (*SQLiteStorage, error) {
// Open database with WAL mode for better concurrency and busy timeout for parallel writes // Open database with WAL mode for better concurrency and busy timeout for parallel writes
// _pragma=busy_timeout(30000) means wait up to 30 seconds for locks instead of failing immediately // _pragma=busy_timeout(30000) means wait up to 30 seconds for locks instead of failing immediately
// Higher timeout helps with parallel issue creation from multiple processes // Higher timeout helps with parallel issue creation from multiple processes
db, err := sql.Open("sqlite", path+"?_journal_mode=WAL&_foreign_keys=ON&_pragma=busy_timeout(30000)") // _time_format=sqlite uses SQLite's native time format for DATETIME columns
db, err := sql.Open("sqlite", path+"?_journal_mode=WAL&_foreign_keys=ON&_pragma=busy_timeout(30000)&_time_format=sqlite")
if err != nil { if err != nil {
return nil, fmt.Errorf("failed to open database: %w", err) return nil, fmt.Errorf("failed to open database: %w", err)
} }