astra/plc-mirror
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix: update lastSuccessTimestamp after each request too

Browse source 
Updating only in run() can be delayed arbitrarily long under load.
This commit is contained in:
Max Ignatenko 2024-11-14 12:34:13 +00:00
parent 2c6a7201ef
commit b26866fe0c
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
cmd/plc-mirror/mirror.go
View file

@ -201,11 +201,12 @@ func (m *Mirror) runOnce(ctx context.Context) error {
return fmt.Errorf("inserting log entry into database: %w", err) return fmt.Errorf("inserting log entry into database: %w", err)
} }
if !lastTimestamp.IsZero() {
m.mu.Lock() m.mu.Lock()
m.lastSuccessTimestamp = time.Now()
if !lastTimestamp.IsZero() {
m.lastRecordTimestamp = lastTimestamp m.lastRecordTimestamp = lastTimestamp
m.mu.Unlock()
} }
m.mu.Unlock()
log.Info().Msgf("Got %d log entries. New cursor: %q", len(newEntries), cursor) log.Info().Msgf("Got %d log entries. New cursor: %q", len(newEntries), cursor)
} }