Skip to content

Commit 3257f05

Browse files
committed
init tagged metrics handler
1 parent 76e1101 commit 3257f05

File tree

1 file changed

+3
-1
lines changed

1 file changed

+3
-1
lines changed

service/history/queues/executable.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,7 @@ func (e *executableImpl) Execute() (retErr error) {
206206
// emit total attempt latency so that we know how much time a task will occpy a worker goroutine
207207
e.taggedMetricsHandler.Timer(metrics.TaskProcessingLatency.GetMetricName()).Record(attemptLatency)
208208

209-
priorityTaggedProvider := e.taggedMetricsHandler.WithTags(metrics.TaskPriorityTag(e.priority.String()))
209+
priorityTaggedProvider := e.taggedMetricsHandler.WithTags(metrics.TaskPriorityTag(executablePriority.String()))
210210
priorityTaggedProvider.Counter(metrics.TaskRequests.GetMetricName()).Record(1)
211211
priorityTaggedProvider.Timer(metrics.TaskScheduleLatency.GetMetricName()).Record(e.scheduleLatency)
212212

@@ -219,6 +219,7 @@ func (e *executableImpl) Execute() (retErr error) {
219219

220220
ns, err := e.namespaceRegistry.GetNamespaceByID(namespace.ID(e.GetNamespaceID()))
221221
if err != nil {
222+
e.taggedMetricsHandler = e.metricsHandler.WithTags(e.estimateTaskMetricTag()...)
222223
return err
223224
}
224225
var callerInfo headers.CallerInfo
@@ -233,6 +234,7 @@ func (e *executableImpl) Execute() (retErr error) {
233234

234235
if !ns.IsOnCluster(e.clusterMetadata.GetCurrentClusterName()) {
235236
// Discard task if the namespace is not on the current cluster.
237+
e.taggedMetricsHandler = e.metricsHandler.WithTags(e.estimateTaskMetricTag()...)
236238
return consts.ErrTaskDiscarded
237239
}
238240

0 commit comments

Comments
 (0)