-
Couldn't load subscription status.
- Fork 485
feat(contrib/sarama): option to add custom consumer/producer span tags #3993
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
base: main
Are you sure you want to change the base?
Conversation
|
Discussing this with colleagues we also came up with the idea of changing this to a more generic |
This sounds more future-proof TBH. |
|
Cool, thanks for the feedback @kakkoyun. I'm on vacation for the next few weeks so not sure when I'll be able to update this, but I'll try to make some time to do so. |
6700372 to
f57982f
Compare
f57982f to
0debdab
Compare
|
@kakkoyun Could you please let me know if there are any additional changes needed, or if the implemented code looks good to you? |
|
@rarguelloF I have pushed the requested changes, can you please check it again? Thanks |
What does this PR do?
Add
WithConsumerCustomTagandWithProducerCustomTagoptions to enable adding custom tags to spans generated by wrapped consumers and producers.Motivation
Including extra information (e.g. Kafka message keys) as tags on spans can greatly enhance observability, enabling users to filter, group, and analyze traces based on specific message properties. This improves debugging capabilities and provides a clearer understanding of message flow within Kafka-based systems.
Reviewer's Checklist
./scripts/lint.shlocally.Unsure? Have a question? Request a review!