This repository has been archived by the owner on May 23, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 345
Breaking change from 0.31 to 0.33 #377
Comments
TextMapAdapter implements full TextMap interface: https://github.com/opentracing/opentracing-java/blob/master/opentracing-api/src/main/java/io/opentracing/propagation/TextMapAdapter.java#L26 |
Why is the override for TextMapAdapter It is an extract from HTTP headers, right? - Shouldn't it be |
@rupinder10 which example are you referring to? |
It was in one of the examples on the opentracing site. Anyway, changing to
TextMapAdapter works.
…On Sun, May 10, 2020 at 12:16 PM Yuri Shkuro ***@***.***> wrote:
The following code doesnt work in 0.33 from the examples
@rupinder10 <https://github.com/rupinder10> which example are you
referring to?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#377 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB56UZYTYGZXQISBP2GI623RQ3HMNANCNFSM4MXSUQVA>
.
|
please propose a PR to fix it |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The following code doesnt work in 0.33 from the examples
tracer.extract(Format.Builtin.HTTP_HEADERS, new TextMapExtractAdapter(headers));
This is because TextMapExtractAdapter no longer implements TextMap in 0.33. So what is the alternate code in 0.33 ?
The text was updated successfully, but these errors were encountered: