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

Document precedence order is multiple context propagators are used #1271

Closed
pavolloffay opened this issue May 25, 2020 · 0 comments · Fixed by #1698
Closed

Document precedence order is multiple context propagators are used #1271

pavolloffay opened this issue May 25, 2020 · 0 comments · Fixed by #1698
Labels
Feature Request Suggest an idea for this project priority:p2 Medium priority issues and bugs. release:required-for-ga Required for 1.0 GA release

Comments

@pavolloffay
Copy link
Member

    ContextPropagators contextPropagators = DefaultContextPropagators.builder()
        .addHttpTextFormat(new JaegerPropagator())
        .addHttpTextFormat(new HttpTraceContext())
        .build();
    OpenTelemetry.setPropagators(contextPropagators);

The DefaultContextPropagators.Builder should document precedence order when multiple context propagation formats are used - especially for extraction. At the moment all formats are extracted and the last one is returned.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Request Suggest an idea for this project priority:p2 Medium priority issues and bugs. release:required-for-ga Required for 1.0 GA release
Projects
None yet
3 participants