From 10f3e4c0dc73075aadae07f9bb58d24a7161b681 Mon Sep 17 00:00:00 2001 From: Roberto Bruggemann Date: Tue, 12 Dec 2017 11:48:55 +0000 Subject: [PATCH] In-memory collector: merge reports after upgrade Upgrading reports individually reduces the amount of upgrades, since some reports might have been upgraded in previously. --- app/collector.go | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/app/collector.go b/app/collector.go index 434edcd81..d71ff2087 100644 --- a/app/collector.go +++ b/app/collector.go @@ -152,7 +152,11 @@ func (c *collector) Report(_ context.Context, timestamp time.Time) (report.Repor c.clean() c.quantise() - rpt := c.merger.Merge(c.reports).Upgrade() + for i := range c.reports { + c.reports[i] = c.reports[i].Upgrade() + } + + rpt := c.merger.Merge(c.reports) c.cached = &rpt return rpt, nil }