Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ddtrace/tracer: don't use UTC() when getting time #1134

Merged
merged 3 commits into from Mar 8, 2022
Merged

ddtrace/tracer: don't use UTC() when getting time #1134

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6db9e3c
ddtrace/tracer/time.now(): Don't call .UTC()
evanj Jan 20, 2022
8a51387
time_windows.go: Make the same change to lowPrecisionNow
evanj Jan 21, 2022
39e56a9
Merge branch 'v1' into evan.jones/time-now
Julio-Guerra Mar 8, 2022
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
5 changes: 3 additions & 2 deletions ddtrace/tracer/time.go
View file
Open in desktop
Expand Up @@ -3,13 +3,14 @@
// This product includes software developed at Datadog (https://www.datadoghq.com/).
// Copyright 2016 Datadog, Inc.

//go:build !windows
// +build !windows

package tracer

import "time"

// now returns current UTC time in nanos.
// now returns the current UNIX time in nanoseconds, as computed by Time.UnixNano().
func now() int64 {
return time.Now().UTC().UnixNano()
return time.Now().UnixNano()
}
2 changes: 1 addition & 1 deletion ddtrace/tracer/time_windows.go
View file
Open in desktop
Expand Up @@ -23,7 +23,7 @@ func highPrecisionNow() int64 {
}

func lowPrecisionNow() int64 {
return time.Now().UTC().UnixNano()
return time.Now().UnixNano()
}

var now func() int64
Expand Down