core/go-store
8
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions 2

refactor(store): clarify compaction lifecycle names
Some checks are pending
Security Scan / security (push) Waiting to run
Details
Test / test (push) Waiting to run
Details

Browse source 
Co-Authored-By: Virgil <virgil@lethean.io>
This commit is contained in:
Virgil 2026-04-04 20:29:21 +00:00
parent c6840745b5
commit 8a117a361d
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
compact.go
View file

@ -137,9 +137,9 @@ func (storeInstance *Store) Compact(options CompactOptions) core.Result {
if !ok {
return core.Result{Value: core.E("store.Compact", "archive file is not writable", nil), OK: false}
}
fileClosed := false
archiveFileClosed := false
defer func() {
if !fileClosed {
if !archiveFileClosed {
_ = file.Close()
}
}()
@ -148,9 +148,9 @@ func (storeInstance *Store) Compact(options CompactOptions) core.Result {
if err != nil {
return core.Result{Value: err, OK: false}
}
writeOK := false
archiveWriteFinished := false
defer func() {
if !writeOK {
if !archiveWriteFinished {
_ = writer.Close()
}
}()
@ -171,11 +171,11 @@ func (storeInstance *Store) Compact(options CompactOptions) core.Result {
if err := writer.Close(); err != nil {
return core.Result{Value: core.E("store.Compact", "close archive writer", err), OK: false}
}
writeOK = true
archiveWriteFinished = true
if err := file.Close(); err != nil {
return core.Result{Value: core.E("store.Compact", "close archive file", err), OK: false}
}
fileClosed = true
archiveFileClosed = true
transaction, err := storeInstance.sqliteDatabase.Begin()
if err != nil {