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

Context api content: Linked list implementation #14

Open
wants to merge 105 commits into
base: master
Choose a base branch
from
Open

Context api content: Linked list implementation #14

wants to merge 105 commits into from

Conversation

satac2
Copy link
Owner

@satac2 satac2 commented Jul 15, 2020

No description provided.

satac2 and others added 30 commits June 8, 2020 14:25
@satac2
demo context header
7867296
@satac2
demo context header
d5a360a
@satac2
added file threadlocal_context.h
5d3b713
@satac2
added fixes
d047ff7
@satac2
Fixed syntatical errors, moved token class definition to context.h an…
f283cc6 
…d added the contextvars_context.h file
@satac2 @0x4b
Update api/include/opentelemetry/context/context.h
e1de13f 
Co-authored-by: Kirk Kelsey <kirkmkelsey@gmail.com>
@satac2
changed define guards and minor syntactical changes
d5b155d
@satac2
removing context vars class
80b2332
@satac2
Merge branch 'context_api' of github.com:satac2/opentelemetry-cpp int…
64cd1c2 
…o context_api
@satac2
removed threadlocal context and made the runtime context methods static
4d24115
@satac2
removed threadlocal context and made the runtime context methods static
a17c028
@satac2
added a constructor
d2df2ac
@satac2
fixed convention issue and removed unnecessary code
c316636
@satac2
Update context.h
7b36945
@satac2
Update context.h
8e02c07
@satac2
context implementation
115cd68
@satac2
Adding tests
0d9fea1
@satac2
Adding tests
565c40e
@satac2
renamed some variables, added tests
2d4a411
@satac2
renamed some variables, added tests
9874c4c
@satac2
Merge branch 'context_api' into context_api_content
7bceaab
@satac2
reversed public/private order
061e564
@satac2
Merge branch 'context_api_content' of github.com:satac2/opentelemetry…
b5c81ad 
…-cpp into context_api_content
@satac2
Added detach test
b2717b3
@satac2
avoiding standard datastructures
b7259cd
@satac2
replaced std::map and created a thread_local context
34eacf2
@satac2
replaced std::map and created a thread_local context
a086993
@satac2
cleaned up
e44e926
@satac2
cleaned up
6edce29
@satac2
fixed undefined behavior in context.h
dff6f8f
satac2 and others added 29 commits July 20, 2020 16:59
@satac2
initialized shared_ptr
c7cbb21
@satac2
removed getValue to try and pass tests
f2542fd
@satac2
restricted GetValue
ba9da8e
@satac2
now deferencing shared_ptr
a9c4eea
@satac2
formatted
a7fbbd0
@satac2
shared Datalist instead of context list
dd3f297
@satac2
one long list
8292012
@satac2
updated some comments
7534cbc
@satac2
removed unnecessary complexity from SetValue
c925962
@satac2
auto formatted
b2e53b2
@satac2
stylistic changes updated ContextValue, add HasKey method
067e7dd
@satac2
removed reference to span_context.h
8396792
@satac2
explicit conversion to contextValue in tests
86b032c
@satac2
explicit type case in tests
f78ab74
@satac2
added nostd::shared_ptr<SpanContext> to ContextValue
77c4c9a
@satac2
removed span_context.h include, so tests would pass, not sure why
f63d595
@satac2
readded span_context.h
5f25cf1
@satac2
re-added span_context include
86eacca
@satac2
testing for the include failure
7a2ea4c
@satac2
removed key_value_iterable_include
961f7ed
@satac2
removed unncessary shared_ptr during iteration
4358d35
@satac2
Merge branch 'master' into context_api_content
c5f6c08
@satac2
added shared_ptr<trace::SpanContext> to context_value
80cbb96
@satac2
Merge remote-tracking branch 'origin/master' into context_api_content
896bc45
@satac2
added shared_ptr<SpanContext> and include
feb3f17
@satac2
Merge branch 'context_api_content' of github.com:satac2/opentelemetry…
acffb61 
…-cpp into context_api_content
@satac2
simplifications
bf94af8
@satac2
removed unecessary includes
5eddd2f
@reyang
Merge branch 'master' into context_api_content
78bd8a6
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0x4b 0x4b 0x4b left review comments

@adityashirodkar adityashirodkar adityashirodkar approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@satac2 @0x4b @adityashirodkar @reyang