Skip to content
This repository has been archived by the owner on Jul 31, 2023. It is now read-only.

Remove call to time.Now() on worker thread when handling record reqs #1210

Merged
merged 1 commit into from
Jun 2, 2020
Merged

Remove call to time.Now() on worker thread when handling record reqs #1210

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion stats/view/worker_commands.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ func (cmd *recordReq) handleCommand(w *worker) {
}
ref := w.getMeasureRef(m.Measure().Name())
for v := range ref.views {
v.addSample(cmd.tm, m.Value(), cmd.attachments, time.Now())
v.addSample(cmd.tm, m.Value(), cmd.attachments, cmd.t)
}
}
}
Expand Down