Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Clarify that optional timestamp means current time should be used. #824

Merged
merged 2 commits into from
Aug 18, 2020

Conversation

anuraaga
Copy link
Contributor

Fixes #796

Changes

Clarifies that when timestamp of event is not provided, current time must be used.

@anuraaga anuraaga requested review from a team and yurishkuro August 18, 2020 04:14
Copy link
Member

@tigrannajaryan tigrannajaryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Co-authored-by: John Watson <jkwatson@gmail.com>
@bogdandrutu
Copy link
Member

Non controversial change, merging

@bogdandrutu bogdandrutu merged commit fa79464 into open-telemetry:master Aug 18, 2020
carlosalberto pushed a commit to carlosalberto/opentelemetry-specification that referenced this pull request Oct 31, 2024
…pen-telemetry#824)

* Clarify that optional timestamp means current time should be used.

* Update specification/trace/api.md

Co-authored-by: John Watson <jkwatson@gmail.com>

Co-authored-by: Bogdan Drutu <lazy@splunk.com>
Co-authored-by: John Watson <jkwatson@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify event timestamp is optional as data or not
7 participants