From dca8a4fb1753ccbe2f885e57fba3a21eb7c74054 Mon Sep 17 00:00:00 2001 From: Cyrill Troxler Date: Thu, 5 Sep 2019 14:41:37 +0200 Subject: [PATCH] Use ticker instead of timer for batch wait Fixes #280 --- pkg/promtail/client/client.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/pkg/promtail/client/client.go b/pkg/promtail/client/client.go index 34526d15af92e..0cf478aadf0b7 100644 --- a/pkg/promtail/client/client.go +++ b/pkg/promtail/client/client.go @@ -109,7 +109,7 @@ func New(cfg Config, logger log.Logger) (Client, error) { func (c *client) run() { batch := map[model.Fingerprint]*logproto.Stream{} batchSize := 0 - maxWait := time.NewTimer(c.cfg.BatchWait) + maxWait := time.NewTicker(c.cfg.BatchWait) defer func() { if len(batch) > 0 { @@ -120,7 +120,6 @@ func (c *client) run() { }() for { - maxWait.Reset(c.cfg.BatchWait) select { case <-c.quit: return