From 6e42ef195b0e28ec2fa1c7c6bbeff4c5517932fc Mon Sep 17 00:00:00 2001 From: Jakob Haahr Taankvist Date: Fri, 9 Feb 2024 08:23:42 +0100 Subject: [PATCH] Removed useless tag There is no need to tag the cachesize with a domain as the cache is host-global, and shared between the domains. --- service/history/workflowcache/cache.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/service/history/workflowcache/cache.go b/service/history/workflowcache/cache.go index 4a635b74dd2..97dc590b401 100644 --- a/service/history/workflowcache/cache.go +++ b/service/history/workflowcache/cache.go @@ -125,7 +125,7 @@ func (c *wfCache) allow(domainID string, workflowID string, rateLimitType rateLi } c.metricsClient. - Scope(metrics.HistoryClientWfIDCacheScope, metrics.DomainTag(domainName)). + Scope(metrics.HistoryClientWfIDCacheScope). UpdateGauge(metrics.WorkflowIDCacheSizeGauge, float64(c.lru.Size())) // Locking is not needed because both getCacheItem and the rate limiter are thread safe