Skip to content

Java tracer v0.36.0 #5793

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

Closed
wants to merge 2 commits into from
Closed

Java tracer v0.36.0 #5793

wants to merge 2 commits into from

Conversation

dougqh
Copy link
Contributor

@dougqh dougqh commented Oct 30, 2019

What does this PR do?

Adds health metric configuration information -- new v0.36.0
Adds Spring Data to integration list -- introduced in v0.33

Motivation

Health Metrics: DataDog/dd-trace-java#1063
Support for Spring Data: DataDog/dd-trace-java#1014 &
DataDog/dd-trace-java#1044

Preview link

https://docs-staging.datadoghq.com/java-tracer-v0.36.0/tracing/setup/java

@dougqh dougqh requested a review from a team as a code owner October 30, 2019 19:19
@@ -37,14 +37,14 @@ Finally, add the following JVM argument when starting your application in your I

**Note**:

* The `-javaagent` needs to be run before the `-jar` file, adding it as a JVM option, not as an application argument. For more information, see the [Oracle documentation][16].
* The `-javaagent` needs to be run before the `-jar` file, adding it as a JVM option, not as an application argument. For more information, see the [Oracle documentation][4].
Copy link
Contributor Author

@dougqh dougqh Oct 30, 2019

Choose a reason for hiding this comment

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

Most of the changes are from a post commit hook -- not part of my change per se.

@@ -209,6 +209,9 @@ The tracer is configured using System Properties and Environment Variables as fo
| `dd.trace.report-hostname` | `DD_TRACE_REPORT_HOSTNAME` | `false` | When enabled, it adds the detected hostname to trace metadata |
| `dd.trace.split-by-tags` | `DD_TRACE_SPLIT_BY_TAGS` | `null` | (Example: `aws.service`) Used to rename spans to be identified with the corresponding service tag |
| `dd.trace.db.client.split-by-instance` | `DD_TRACE_DB_CLIENT_SPLIT_BY_INSTANCE` | `false` | When set to `true` db spans get assigned the instance name as the service name |
| `dd.trace.health.metrics.enabled` | `DD_TRACE_HEALTH_METRICS_ENABLED` | `false` | When set to `true` sends tracer health metrics |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

health metric additions

@@ -174,14 +174,15 @@ Don't see your desired datastores? Datadog is continually adding additional supp
| Dropwizard Views | 0.7+ | Fully Supported | `dropwizard`, `dropwizard-view` |
| Hystrix | 1.4+ | Fully Supported | `hystrix` |
| Twilio SDK | 0+ | Fully Supported | `twilio-sdk` |
| Spring Data | 1.8+ | Fully Supported | `spring-data` |
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Spring Data addition

@dougqh
Copy link
Contributor Author

dougqh commented Oct 30, 2019

Closing, I'll resubmit with a proper branch name.

@dougqh dougqh closed this Oct 30, 2019
@dougqh dougqh deleted the java-tracer-v0.36.0 branch October 30, 2019 20:25
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.

1 participant