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

Context propagation update (OTEP 66) #720

Merged
merged 47 commits into from
Mar 20, 2020
Merged
Changes from 1 commit
Commits
Show all changes
47 commits
Select commit Hold shift + click to select a range
e2f642b
Initial Propagators refactor.
carlosalberto Dec 16, 2019
d5d3e02
Add tests for the new propagators changes.
carlosalberto Dec 17, 2019
b60b02d
Make the SDK compile/pass.
carlosalberto Dec 17, 2019
9160417
Make the OT shim compile/pass.
carlosalberto Dec 17, 2019
b97d700
Make contrib components compile/pass.
carlosalberto Dec 17, 2019
0b109f9
Improvement over Span/SpanContext handling in Context.
carlosalberto Dec 18, 2019
51732a9
Add Span.setParent(Context) overload.
carlosalberto Dec 19, 2019
d6894d4
Do not provide default values for Span/SpanContext keys in Context.
carlosalberto Dec 19, 2019
d5acde5
Improve the Context's active state handling.
carlosalberto Dec 19, 2019
71f04d4
Rename DistributedContext to CorrelationContext.
carlosalberto Dec 19, 2019
eb1e411
Improve names for correlationcontext's ContextUtils methods.
carlosalberto Dec 19, 2019
bbfa035
Don't provide an automatic default for current CorrelationContext.
carlosalberto Dec 19, 2019
b9f9b93
Improve the client-server example after the recent changes.
carlosalberto Dec 19, 2019
fdf644b
Adds CorrelationContext.Builder.setParent(Context) overload.
carlosalberto Dec 20, 2019
fd2b4a9
s/be/become.
carlosalberto Dec 20, 2019
24fecd1
Fix javadoc.
carlosalberto Dec 20, 2019
3f7c227
Merge branch 'master' into context_prop_update
carlosalberto Jan 9, 2020
b56ac60
No need to use diamond.
carlosalberto Jan 9, 2020
08297c6
Simply import withScopedContext().
carlosalberto Jan 9, 2020
be3dd73
Merge branch 'master' into context_prop_update
carlosalberto Jan 22, 2020
e06c32f
Fix the API/SDK build.
carlosalberto Jan 24, 2020
4c48266
Remove the builder from the Propagators interface.
carlosalberto Jan 24, 2020
1328ec5
Fix name.
carlosalberto Jan 24, 2020
d35f19e
Use Collections.emptyList() directly.
carlosalberto Jan 24, 2020
14acd5c
Rename Propagators to ContextPropagators.
carlosalberto Jan 24, 2020
b0649da
Move context/ members in api/ to context_prop/
carlosalberto Feb 20, 2020
762408e
Add check/tests for null scoped Contexts.
carlosalberto Feb 20, 2020
05b7628
Rename ContextUtils classes to better alternatives.
carlosalberto Feb 21, 2020
1113f61
Update the context* util classes.
carlosalberto Mar 9, 2020
d271bb6
Merge branch 'master' into context_prop_update
carlosalberto Mar 10, 2020
43e42d4
Make the code compile after the latest master merge.
carlosalberto Mar 10, 2020
014b3c4
Cache the fields() in our default composite propagator.
carlosalberto Mar 10, 2020
f3d29a3
Remove the overloads of setParent(Context) for now.
carlosalberto Mar 10, 2020
555095b
Use DefaultSpan for the tests instead of calling getTracerProvider()
carlosalberto Mar 10, 2020
3c1f781
Fix the sdk testbed artifact build.
carlosalberto Mar 10, 2020
64b5b1e
Merge branch 'master' into context_prop_update
carlosalberto Mar 13, 2020
136f0aa
Make the B3 propagator comply with the new propagator API.
carlosalberto Mar 14, 2020
ddc370a
Simplify the HttpTraceContextTest tests.
carlosalberto Mar 14, 2020
a1d93ec
Simplify the ContextUtils* classes handling of default values.
carlosalberto Mar 14, 2020
3c96627
Minor nit.
carlosalberto Mar 14, 2020
5f820cd
Merge branch 'master' into context_prop_update
carlosalberto Mar 15, 2020
9a1eb5b
Update api/src/main/java/io/opentelemetry/OpenTelemetry.java
carlosalberto Mar 17, 2020
268b546
Annotate ContextPropagators with ThreadSafe instead of Immutable.
carlosalberto Mar 19, 2020
a0ec628
Do not use the fully qualified ContextUtils identifier.
carlosalberto Mar 19, 2020
7a3cbeb
Remove SpanContext support from TracingContextUtils.
carlosalberto Mar 20, 2020
88fdcbb
Rever to using non-defaulted key for TracingContextUtils.
carlosalberto Mar 20, 2020
37ba00b
Revert the default keys for CorrelationsContextUtils.
carlosalberto Mar 20, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Move context/ members in api/ to context_prop/
  • Loading branch information
carlosalberto committed Feb 20, 2020
commit b0649dacf2038a1c1f94e9917ca89d3bd3958404
Original file line number Diff line number Diff line change
@@ -17,7 +17,6 @@
package io.opentelemetry.context.propagation;

import io.grpc.Context;
import io.opentelemetry.internal.Utils;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
@@ -85,7 +84,10 @@ public static final class Builder {
* @since 0.3.0
*/
public Builder addHttpTextFormat(HttpTextFormat textFormat) {
Utils.checkNotNull(textFormat, "textFormat");
if (textFormat == null) {
throw new NullPointerException("textFormat");
}
bogdandrutu marked this conversation as resolved.
Show resolved Hide resolved

textPropagators.add(textFormat);
return this;
}