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

Added kafka context propagation note #8194

Merged
merged 4 commits into from
Aug 31, 2020
Merged

Conversation

andrewsouthard1
Copy link
Contributor

@andrewsouthard1 andrewsouthard1 commented Aug 5, 2020

Added configuration specific to Kafka from here: DataDog/dd-trace-java#1448

What does this PR do?

Motivation

Preview link

https://docs-staging.datadoghq.com/andrewsouthard1-patch-1/tracing/compatibility_requirements/java/#networking-framework-compatibility

Check preview base path using the URL in details in preview status check.

Additional Notes

Added configuration specific to Kafka from here: DataDog/dd-trace-java#1448
@andrewsouthard1 andrewsouthard1 requested a review from a team as a code owner August 5, 2020 14:28
@github-actions github-actions bot added the tracing Content changed in the tracing folder label Aug 5, 2020
@@ -87,6 +87,8 @@ Don't see your desired web frameworks? Datadog is continually adding additional
| Rabbit AMQP | 2.7+ | Fully Supported | `amqp`, `rabbitmq` |
| Spring WebClient | 5.0+ | Fully Supported | `spring-webflux`, `spring-webflux-client` |

**Note**: Our Kafka Integration works with version `0.11+` as this version of Kafka supports the Header API, where we inject/extract trace context. If you a running a mixed version environment, the Kafka broker will sometimes report the newer version of Kafka. This can cause an issue as the tracer will try to inject headers when it’s not supported by the local producer, or older consumers will be unable to consume the message because of the presence of headers. To prevent this issue if you are running a mixed version Kafka environment with versions older than 0.11, context propagation can be disabled with the following environment variable: `DD_KAFKA_CLIENT_PROPAGATION_ENABLED=false`.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"If you are running"

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"sometimes incorrectly report the newer version of Kafka"

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Co-authored-by: ruthnaebeck <19349244+ruthnaebeck@users.noreply.github.com>
@apigirl apigirl merged commit dc368bf into master Aug 31, 2020
@apigirl apigirl deleted the andrewsouthard1-patch-1 branch August 31, 2020 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tracing Content changed in the tracing folder
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants