Skip to content

Commit

Permalink
profiler: set default CPU profiling interval to match duration
Browse files Browse the repository at this point in the history
After testing this default on many high-volume internal workloads, we've
determined this default is safe for production. It's also important for
the code hotspots feature to be able to associated as many traces as
possible with profiling data.
  • Loading branch information
nsrip-dd committed Feb 15, 2022
1 parent d4e3832 commit 4ad8f3f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion profiler/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const (
DefaultPeriod = time.Minute

// DefaultDuration specifies the default length of the CPU profile snapshot.
DefaultDuration = time.Second * 15
DefaultDuration = time.Minute

// DefaultUploadTimeout specifies the default timeout for uploading profiles.
// It can be overwritten using the DD_PROFILING_UPLOAD_TIMEOUT env variable
Expand Down

0 comments on commit 4ad8f3f

Please sign in to comment.