Fix metrics scope misuse (temporalio#2515)

* Fix metrics scope bug when emitting mutable state metrics
jleach4 · Feb 17, 2022 · ed29a48 · ed29a48
1 parent 1f2b291
commit ed29a48
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/service/history/workflow/metrics.go b/service/history/workflow/metrics.go
@@ -68,18 +68,18 @@ func emitMutableStateStatus(
 	sizeScope.RecordDistribution(metrics.ChildInfoSize, stats.ChildInfoSize)
 	countScope.RecordDistribution(metrics.ChildInfoCount, stats.ChildInfoCount)
 
-	countScope.RecordDistribution(metrics.RequestCancelInfoSize, stats.RequestCancelInfoSize)
+	sizeScope.RecordDistribution(metrics.RequestCancelInfoSize, stats.RequestCancelInfoSize)
 	countScope.RecordDistribution(metrics.RequestCancelInfoCount, stats.RequestCancelInfoCount)
 
 	sizeScope.RecordDistribution(metrics.SignalInfoSize, stats.SignalInfoSize)
 	countScope.RecordDistribution(metrics.SignalInfoCount, stats.SignalInfoCount)
 
 	sizeScope.RecordDistribution(metrics.BufferedEventsSize, stats.BufferedEventsSize)
-	sizeScope.RecordDistribution(metrics.BufferedEventsCount, stats.BufferedEventsCount)
+	countScope.RecordDistribution(metrics.BufferedEventsCount, stats.BufferedEventsCount)
 
 	if stats.HistoryStatistics != nil {
 		sizeScope.RecordDistribution(metrics.HistorySize, stats.HistoryStatistics.SizeDiff)
-		sizeScope.RecordDistribution(metrics.HistoryCount, stats.HistoryStatistics.CountDiff)
+		countScope.RecordDistribution(metrics.HistoryCount, stats.HistoryStatistics.CountDiff)
 	}
 }