Merge pull request #43 from starskey-io/starskey-alpha-30

pager sync comments
starskey-io · Jan 22, 2025 · 156b6bf · 156b6bf
2 parents ae8a168 + 0d06c64
commit 156b6bf
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/pager/pager.go b/pager/pager.go
@@ -115,9 +115,9 @@ func (p *Pager) backgroundSync() {
 	for {
 		select {
 		case <-ticker.C:
-			p.file.Sync()
+			_ = p.file.Sync()
 		case <-p.syncQuit:
-			p.file.Sync() // Escalate then return..
+			_ = p.file.Sync() // Escalate then return..
 			return
 		}
 	}
@@ -358,7 +358,7 @@ func (p *Pager) FileName() string {
 	return p.file.Name()
 }
 
-// EscalateFSync escalates an fsync to disk sending a trigger through channel
+// EscalateFSync escalates a disk fsync
 func (p *Pager) EscalateFSync() {
-	p.file.Sync()
+	_ = p.file.Sync() // Is thread safe
 }