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.

Sign up for GitHub

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

Jump to bottom

The instances of SpanContext should be immutable #1002

Closed
codeboten opened this issue Aug 17, 2020 · 1 comment
Closed

The instances of SpanContext should be immutable #1002

codeboten opened this issue Aug 17, 2020 · 1 comment
Assignees
@JasonXZLiu
Labels
api Affects the API package. good first issue Good first issue help wanted release:required-for-ga To be resolved before GA release sdk Affects the SDK package.

Comments

@codeboten
Copy link
Contributor

To be compliant with the tracing spec, the instances of SpanContext should be immutable

https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/api.md#spancontext
@codeboten codeboten added good first issue Good first issue api Affects the API package. sdk Affects the SDK package. help wanted release:required-for-ga To be resolved before GA release labels Aug 17, 2020
@ocelotl ocelotl mentioned this issue Aug 17, 2020
Closed
@JasonXZLiu
Copy link
Member

Hi! Is this issue free to pick up? I would love to work on it! -cc @alolita

@JasonXZLiu JasonXZLiu mentioned this issue Sep 17, 2020
Closed
5 tasks
@JasonXZLiu JasonXZLiu mentioned this issue Sep 17, 2020
Merged
5 tasks
@lzchen lzchen closed this as completed Oct 7, 2020
srikanthccv pushed a commit to srikanthccv/opentelemetry-python that referenced this issue Nov 1, 2020
@ivansenic
fix(opentelemetry-plugin-xml-http-request): define span kind as CLIEN…
6cf5406 
…T for xmlhttprequests (open-telemetry#1002)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JasonXZLiu JasonXZLiu

Labels
api Affects the API package. good first issue Good first issue help wanted release:required-for-ga To be resolved before GA release sdk Affects the SDK package.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@codeboten @lzchen @JasonXZLiu