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

Commit 99162e4

Browse files
authored
Metrics/TimeSeries: start time should not be included while end time should. (#142)
1 parent bbcfad0 commit 99162e4

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/opencensus/proto/metrics/v1/metrics.proto

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -126,8 +126,8 @@ message LabelKey {
126126
// of a metric.
127127
message TimeSeries {
128128
// Must be present for cumulative metrics. The time when the cumulative value
129-
// was reset to zero. The cumulative value is over the time interval
130-
// [start_timestamp, timestamp). If not specified, the backend can use the
129+
// was reset to zero. Exclusive. The cumulative value is over the time interval
130+
// (start_timestamp, timestamp]. If not specified, the backend can use the
131131
// previous recorded value.
132132
google.protobuf.Timestamp start_timestamp = 1;
133133

@@ -151,8 +151,8 @@ message LabelValue {
151151

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

158158
// The actual point value.

0 commit comments

Comments
 (0)