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

Commit

Permalink
Metrics/TimeSeries: start time should not be included while end time …
Browse files Browse the repository at this point in the history
…should. (#142)
  • Loading branch information
songy23 authored Oct 25, 2018
1 parent bbcfad0 commit 99162e4
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions src/opencensus/proto/metrics/v1/metrics.proto
Original file line number Diff line number Diff line change
Expand Up @@ -126,8 +126,8 @@ message LabelKey {
// of a metric.
message TimeSeries {
// Must be present for cumulative metrics. The time when the cumulative value
// was reset to zero. The cumulative value is over the time interval
// [start_timestamp, timestamp). If not specified, the backend can use the
// was reset to zero. Exclusive. The cumulative value is over the time interval
// (start_timestamp, timestamp]. If not specified, the backend can use the
// previous recorded value.
google.protobuf.Timestamp start_timestamp = 1;

Expand All @@ -151,8 +151,8 @@ message LabelValue {

// A timestamped measurement.
message Point {
// The moment when this point was recorded. If not specified, the timestamp
// will be decided by the backend.
// The moment when this point was recorded. Inclusive.
// If not specified, the timestamp will be decided by the backend.
google.protobuf.Timestamp timestamp = 1;

// The actual point value.
Expand Down

0 comments on commit 99162e4

Please sign in to comment.