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

+kafka: permit specifying custom propagation implementation #1292

Merged

Conversation

hughsimpson
Copy link
Contributor

No description provided.

@hughsimpson hughsimpson changed the title +kamon: permit specifying custom propagation implementation +kafka: permit specifying custom propagation implementation Aug 21, 2023
@yarosman
Copy link
Contributor

yarosman commented Aug 28, 2023

relates to #1284

it would be great to add supporting of b3 format as one of the defaults.

@hughsimpson
Copy link
Contributor Author

hughsimpson commented Aug 30, 2023

@yarosman I think b3 should be pretty similar to the custom implementation I made for the test -- if you're up for implementing it, I'll happily accept a pr to my branch (although I don't need it myself, so tbh probably better to just open separately)

@yarosman
Copy link
Contributor

@ivantopo What do you think about merging it ?

@hughsimpson
Copy link
Contributor Author

Think Ivan's a bit busy with other stuff at the moment so might take a while to get to this - I've been overriding the class locally in the meantime

@hughsimpson hughsimpson merged commit 0539e62 into kamon-io:master Oct 3, 2023
1 check passed
@hughsimpson hughsimpson deleted the configurable_kafka_propagation branch October 3, 2023 09:53
@hughsimpson
Copy link
Contributor Author

@yarosman released as 2.6.4, lemme know how it goes

@yarosman
Copy link
Contributor

yarosman commented Oct 6, 2023

@yarosman released as 2.6.4, lemme know how it goes

I hope I will try to integrate it on next week ))

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

Successfully merging this pull request may close these issues.

2 participants