Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

Commit

Permalink
Change default level to simple and move FD metrics to moderate level
Browse files Browse the repository at this point in the history
  • Loading branch information
xichen2020 committed Sep 27, 2017
1 parent 6e9713e commit 133a797
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions instrument/extended.go
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,12 @@ const (
// SimpleExtendedMetrics describes just a simple level of extended metrics:
// - number of active goroutines
// - number of configured gomaxprocs
// - number of file descriptors
SimpleExtendedMetrics

// ModerateExtendedMetrics describes a moderately verbose level of extended metrics:
// - number of file descriptors
ModerateExtendedMetrics

// DetailedExtendedMetrics describes a detailed level of extended metrics:
// - number of active goroutines
// - number of configured gomaxprocs
Expand All @@ -60,13 +63,14 @@ const (
DetailedExtendedMetrics

// DefaultExtendedMetricsType is the default extended metrics level.
DefaultExtendedMetricsType = DetailedExtendedMetrics
DefaultExtendedMetricsType = SimpleExtendedMetrics
)

var (
validExtendedMetricsTypes = []ExtendedMetricsType{
NoExtendedMetrics,
SimpleExtendedMetrics,
ModerateExtendedMetrics,
DetailedExtendedMetrics,
}
)
Expand All @@ -77,6 +81,8 @@ func (t ExtendedMetricsType) String() string {
return "none"
case SimpleExtendedMetrics:
return "simple"
case ModerateExtendedMetrics:
return "moderate"
case DetailedExtendedMetrics:
return "detailed"
}
Expand Down Expand Up @@ -225,7 +231,7 @@ func (r *ExtendedMetricsReporter) reportRuntimeMetrics() {

r.runtime.NumGoRoutines.Update(float64(runtime.NumGoroutine()))
r.runtime.GoMaxProcs.Update(float64(runtime.GOMAXPROCS(0)))
if r.metricsType == SimpleExtendedMetrics {
if r.metricsType < DetailedExtendedMetrics {
return
}

Expand Down Expand Up @@ -254,7 +260,7 @@ func (r *ExtendedMetricsReporter) reportRuntimeMetrics() {
}

func (r *ExtendedMetricsReporter) reportProcessMetrics() {
if r.metricsType == NoExtendedMetrics {
if r.metricsType < ModerateExtendedMetrics {
return
}
numFDs, err := process.NumFDs(r.process.pid)
Expand Down

0 comments on commit 133a797

Please sign in to comment.