Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Breaking change from 0.31 to 0.33 #377

Open
rupinder10 opened this issue May 2, 2020 · 6 comments
Open

Breaking change from 0.31 to 0.33 #377

rupinder10 opened this issue May 2, 2020 · 6 comments

Comments

@rupinder10
Copy link

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 ?

@yurishkuro
Copy link
Member

@asad-awadia
Copy link

Why is the override for TextMapAdapter put instead of a get

It is an extract from HTTP headers, right? - Shouldn't it be getting the B3 headers from the request? instead of putting stuff in the map?

@yurishkuro
Copy link
Member

The following code doesnt work in 0.33 from the examples

@rupinder10 which example are you referring to?

@rupinder10
Copy link
Author

rupinder10 commented May 10, 2020 via email

@asad-awadia
Copy link

@yurishkuro
Copy link
Member

please propose a PR to fix it

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants