Skip to content

Commit f079042

Browse files
authored
Merge pull request #41 from caseyho/div-by-zero-event-error
Avoid dividing by zero in rare case where zero events are read before logger is called
2 parents ea96859 + 0fbefb1 commit f079042

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

cmd/client/main.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ func main() {
5252
case <-ticker.C:
5353
eventsRead := c.EventsRead.Load()
5454
bytesRead := c.BytesRead.Load()
55-
avgEventSize := bytesRead / eventsRead
55+
avgEventSize := bytesRead / max(1, eventsRead)
5656
logger.Info("stats", "events_read", eventsRead, "bytes_read", bytesRead, "avg_event_size", avgEventSize)
5757
}
5858
}

0 commit comments

Comments
 (0)