Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add ability to set explicit start & end time for span #134

Closed
griels opened this issue Sep 11, 2019 · 3 comments · Fixed by #179
Closed

Add ability to set explicit start & end time for span #134

griels opened this issue Sep 11, 2019 · 3 comments · Fixed by #179
Labels
api Affects the API package. good first issue Good first issue tracing

Comments

@griels
Copy link

griels commented Sep 11, 2019

This is present in other implementations.
For example, Java has spanBuilder.withStartTimestamp.

It is crucial for situations where the start time is only known after an event has completed, for example when marshalling a span from a 'foreign' tracing system (this is my situation).

@Oberon00
Copy link
Member

Spec reference: https://github.com/open-telemetry/oteps/blob/master/text/0002-remove-spandata.md

@griels
Copy link
Author

griels commented Sep 11, 2019

Great, sounds like it's on the todo list :) Thanks.

@johananl
Copy link
Contributor

Same for setting an explicit end time when calling finish() on a span. I guess we can handle both in this issue.

@Oberon00 Oberon00 changed the title Add ability to set start time for span Add ability to set explicit start & end time for span Sep 25, 2019
@c24t c24t added the good first issue Good first issue label Sep 25, 2019
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this issue Nov 1, 2020
* docs: improve readme

* docs: improve readme
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api Affects the API package. good first issue Good first issue tracing
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants