Commit c163130
feat(om2): clarify Timestamp and Start Timestamp semantics (#2929)
* feat(om2): clarify Timestamp and Start Timestamp semantics
- Add paragraph to Timestamps section distinguishing explicit Timestamp
from Start Timestamp and clarifying naming conventions
- Expand Sample section to explain what Timestamp and Start Timestamp
represent, and how Start Timestamp helps detect counter resets
- Remove redundant Start Timestamp explanations from Counter and
Histogram sections since they are now covered in Sample
- Fix "timestamp of the reset" -> "approximate reset time" in Counter
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Coded with Claude Sonnet 4.6.
* feat(om2): strengthen Start Timestamp normative language
- "specifies" -> "SHOULD specify" to allow edge cases where exact
measurement start is not known
- "is equivalent to" -> "MUST be treated as equivalent to" to make
zero Start Timestamp handling a normative requirement on ingestors
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Coded with Claude Sonnet 4.6.
* fix(om2): remove redundant wording from Summary Start Timestamp
Align with Counter and Histogram sections by using "SHOULD have a
Start Timestamp" consistently.
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Coded with Claude Sonnet 4.6.
* Apply suggestion from @dashpole
Co-authored-by: David Ashpole <dashpole@google.com>
Signed-off-by: George Krajcsovits <krajorama@users.noreply.github.com>
* Apply suggestions from code review
Co-authored-by: Bartlomiej Plotka <bwplotka@gmail.com>
Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>
---------
Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
Signed-off-by: George Krajcsovits <krajorama@users.noreply.github.com>
Signed-off-by: Bartlomiej Plotka <bwplotka@gmail.com>
Co-authored-by: David Ashpole <dashpole@google.com>
Co-authored-by: Bartlomiej Plotka <bwplotka@gmail.com>1 parent 3d727c5 commit c163130
1 file changed
Lines changed: 14 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
152 | 152 | | |
153 | 153 | | |
154 | 154 | | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
155 | 161 | | |
156 | 162 | | |
157 | 163 | | |
| |||
186 | 192 | | |
187 | 193 | | |
188 | 194 | | |
189 | | - | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
190 | 198 | | |
191 | | - | |
| 199 | + | |
192 | 200 | | |
193 | 201 | | |
194 | 202 | | |
| |||
262 | 270 | | |
263 | 271 | | |
264 | 272 | | |
265 | | - | |
| 273 | + | |
266 | 274 | | |
267 | | - | |
| 275 | + | |
268 | 276 | | |
269 | 277 | | |
270 | 278 | | |
| |||
314 | 322 | | |
315 | 323 | | |
316 | 324 | | |
317 | | - | |
| 325 | + | |
318 | 326 | | |
319 | 327 | | |
320 | 328 | | |
| |||
416 | 424 | | |
417 | 425 | | |
418 | 426 | | |
419 | | - | |
| 427 | + | |
420 | 428 | | |
421 | 429 | | |
422 | 430 | | |
| |||
0 commit comments