Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Add ObservedTimestamp Field #367

Closed
1 of 4 tasks
djaglowski opened this issue Feb 9, 2022 · 0 comments · Fixed by #370
Closed
1 of 4 tasks

Add ObservedTimestamp Field #367

djaglowski opened this issue Feb 9, 2022 · 0 comments · Fixed by #370
Assignees

Comments

@djaglowski
Copy link
Member

The notion of an Observed Timestamp is a recent addition to the Logs Data Model.

This field should be added to the library and populated automatically as appropriate.

  • Add ObservedTimestamp field to entry.Entry
  • entry.New should initialize ObservedTimestamp, but not Timestamp
  • All usages of Timestamp must be evaluated. (i.e. what happens if not set?)
  • Ensure all preexisting assignments to Timestamp are still valid (excepting entry.New initialization)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant