From 3cde9287a65ca7cc94060befeb3f83cdca8eeb21 Mon Sep 17 00:00:00 2001 From: Bogdan Drutu Date: Wed, 7 Sep 2022 15:01:01 -0700 Subject: [PATCH] scrape: remove unused member from cacheEntry (#11281) Signed-off-by: Bogdan Drutu --- scrape/scrape.go | 9 ++++----- scrape/scrape_test.go | 4 +--- 2 files changed, 5 insertions(+), 8 deletions(-) diff --git a/scrape/scrape.go b/scrape/scrape.go index 033aa46852..a84e8cbea0 100644 --- a/scrape/scrape.go +++ b/scrape/scrape.go @@ -845,7 +845,6 @@ type cacheEntry struct { lastIter uint64 hash uint64 lset labels.Labels - meta metadata.Metadata } type scrapeLoop struct { @@ -991,11 +990,11 @@ func (c *scrapeCache) get(met string) (*cacheEntry, bool) { return e, true } -func (c *scrapeCache) addRef(met string, ref storage.SeriesRef, lset labels.Labels, meta metadata.Metadata, hash uint64) { +func (c *scrapeCache) addRef(met string, ref storage.SeriesRef, lset labels.Labels, hash uint64) { if ref == 0 { return } - c.series[met] = &cacheEntry{ref: ref, lastIter: c.iter, lset: lset, meta: meta, hash: hash} + c.series[met] = &cacheEntry{ref: ref, lastIter: c.iter, lset: lset, hash: hash} } func (c *scrapeCache) addDropped(met string) { @@ -1609,7 +1608,7 @@ loop: // Bypass staleness logic if there is an explicit timestamp. sl.cache.trackStaleness(hash, lset) } - sl.cache.addRef(mets, ref, lset, meta, hash) + sl.cache.addRef(mets, ref, lset, hash) if sampleAdded && sampleLimitErr == nil { seriesAdded++ } @@ -1835,7 +1834,7 @@ func (sl *scrapeLoop) addReportSample(app storage.Appender, s string, t int64, v switch errors.Cause(err) { case nil: if !ok { - sl.cache.addRef(s, ref, lset, metadata.Metadata{}, lset.Hash()) + sl.cache.addRef(s, ref, lset, lset.Hash()) } return nil case storage.ErrOutOfOrderSample, storage.ErrDuplicateSampleForTimestamp: diff --git a/scrape/scrape_test.go b/scrape/scrape_test.go index f8c43bd7f8..e9b10e8066 100644 --- a/scrape/scrape_test.go +++ b/scrape/scrape_test.go @@ -39,7 +39,6 @@ import ( "github.com/prometheus/prometheus/discovery/targetgroup" "github.com/prometheus/prometheus/model/exemplar" "github.com/prometheus/prometheus/model/labels" - "github.com/prometheus/prometheus/model/metadata" "github.com/prometheus/prometheus/model/relabel" "github.com/prometheus/prometheus/model/textparse" "github.com/prometheus/prometheus/model/timestamp" @@ -1556,13 +1555,12 @@ func TestScrapeLoopAppendCacheEntryButErrNotFound(t *testing.T) { require.NoError(t, warning) var lset labels.Labels - var meta metadata.Metadata p.Next() mets := p.Metric(&lset) hash := lset.Hash() // Create a fake entry in the cache - sl.cache.addRef(mets, fakeRef, lset, meta, hash) + sl.cache.addRef(mets, fakeRef, lset, hash) now := time.Now() slApp := sl.appender(context.Background())