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

Commit

Permalink
Fix more comments.
Browse files Browse the repository at this point in the history
  • Loading branch information
bogdandrutu committed Sep 18, 2018
1 parent f06b699 commit 1673321
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions src/opencensus/proto/metrics/v1/metrics.proto
Original file line number Diff line number Diff line change
Expand Up @@ -93,11 +93,11 @@ message MetricDescriptor {
// then the start_time should also be reset.
CUMULATIVE_DISTRIBUTION = 5;

// Old libraries implemented Histograms as a summary samples observations
// Some frameworks implemented Histograms as a summary of observations
// (usually things like request durations and response sizes). While it
// also provides a total count of observations and a sum of all observed
// values, it calculates configurable quantiles over a sliding time window.
// This is not recommended, since cannot be aggregated.
// This is not recommended, since it cannot be aggregated.
SUMMARY = 7;
}
Type type = 4;
Expand Down Expand Up @@ -159,7 +159,7 @@ message Point {
// A distribution value.
DistributionValue distribution_value = 4;

// A summary value. This is not recommended, since cannot be aggregated.
// A summary value. This is not recommended, since it cannot be aggregated.
SummaryValue summary_value = 5;
}
}
Expand Down

0 comments on commit 1673321

Please sign in to comment.