-
Notifications
You must be signed in to change notification settings - Fork 704
Adding Context API #395
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
Merged
Merged
Adding Context API #395
Changes from 54 commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
1500bec
Context API
3bdf1c8
Lint fixes
e144871
Merge remote-tracking branch 'origin/master' into adding-context-api
569878e
adding support for python 3.4 via thread-local
55c817a
Fix lint
746e591
review feedback
b53bdea
fix docs build
76a8c03
adding named tracer support back
3fe5a23
mypy fixes
1f8d67f
more mypy fixes
4bdd27c
lint fixes
b28fb3f
moving contextvars and threadlocal context to sdk
51af769
adding documentation
530c21a
documentation
16eb703
replace get_value w/ context.get_value
ddd81a5
log as error. more cleanup
b935bf5
zipkin exporter needs the sdk
15805a4
small cleanups
68b98e5
don't bother constructing a map for entry_points
3a2b0ef
cleaning up more code, adding some context tests
62c76ec
mypy fixes
5dca7cc
Fix ThreadlocalContext, use ContextVarsContext for tests with python …
7203e72
fix tests
e43f168
as per review feedback, removing use
f81a673
adding tests for sdk threadlocal and contextvar contexts
ba65845
removing create_key
ed187bf
add try/except
2b8c69f
adding tests for threads and asyncio
1a0217f
filling in asyncio test
212b14b
Test
ocelotl dbfeb98
Revert "Test"
ocelotl 2235bc3
fix threadlocal behaviour, more feedback changes
5dc9fd5
fix context restore bug
02c7c4c
simplifying code, removing unnecessary methods
ae2cbc4
cleaning up docs
6f99621
Merge branch 'master' into adding-context-api
codeboten 01e0054
Splitting the Context interface
ebcf0bd
update RuntimeContext when setting current
ddf3c8d
Merge remote-tracking branch 'origin/master' into adding-context-api
6f9780b
add set_current/get_current on RuntimeContext to store current Contex…
49b6abd
cleaning up mypy and fixing 3.4 tests
4c5083a
cleaning up tests to use context api interface
5a8a9fb
fix mypy
d5da10c
fix 3.4 tests
7995ac3
fixing docs
dbbdefb
Merge remote-tracking branch 'origin/master' into adding-context-api
37f1555
return the old context when calling set_current to allow for restore
81f06a9
mypy fixes
f29111a
removing snapshot from runtime context
057c22c
Merge remote-tracking branch 'origin/master' into adding-context-api
5376e60
cleaning up unused methods
e7d0286
lint fixes
0cee462
further sinmplifying RuntimeContext
f8032c1
lint fix
02b8f71
Update opentelemetry-api/src/opentelemetry/context/__init__.py
codeboten 727ae50
adding test, updating copyright on new files
c9c66f3
Merge branch 'adding-context-api' of github.com:codeboten/opentelemet…
f25992f
Removing unnecessary copy
91d2ebc
set default for contextvars
335678e
feedback from review
619fc1b
minor fix
1f9bd76
review feedback
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
2 changes: 1 addition & 1 deletion
2
docs/opentelemetry.context.base_context.rst → docs/opentelemetry.context.context.rst
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,7 @@ | ||
opentelemetry.context.base\_context module | ||
========================================== | ||
|
||
.. automodule:: opentelemetry.context.base_context | ||
.. automodule:: opentelemetry.context.context | ||
:members: | ||
:undoc-members: | ||
:show-inheritance: |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
45 changes: 0 additions & 45 deletions
45
opentelemetry-api/src/opentelemetry/context/async_context.py
This file was deleted.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.