Skip to content

Conversation

@trask
Copy link
Member

@trask trask commented Aug 22, 2025

Resolves #14485

@otelbot-java-instrumentation
Copy link
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@trask trask marked this pull request as ready for review August 22, 2025 02:14
@trask trask requested a review from a team as a code owner August 22, 2025 02:14
Copy link
Contributor

@laurit laurit left a comment

Choose a reason for hiding this comment

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

also has the same code

StringWriter writer = new StringWriter();
throwable.printStackTrace(new PrintWriter(writer));
attributes.put(ExceptionAttributes.EXCEPTION_STACKTRACE, writer.toString());
if (builder instanceof ExtendedLogRecordBuilder) {
Copy link
Contributor

Choose a reason for hiding this comment

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

https://github.com/open-telemetry/opentelemetry-java-instrumentation/blob/main/conventions/src/main/kotlin/io.opentelemetry.instrumentation.library-instrumentation.gradle.kts adds dependency to instrumentation-api-incubator that pulls in opentelemetry-api-incubator so ExtendedLogRecordBuilder is always present. Really this instrumentation doesn't use the instrumentation api.

@trask trask force-pushed the support-custom-exception-handling branch 2 times, most recently from 380f60f to 2e553f7 Compare August 22, 2025 15:38
@otelbot-java-instrumentation
Copy link
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@trask trask force-pushed the support-custom-exception-handling branch from 4544a29 to 40c7dd3 Compare August 22, 2025 15:44
@otelbot-java-instrumentation
Copy link
Contributor

🔧 The result from spotlessApply was committed to the PR branch.

@laurit laurit merged commit c607938 into open-telemetry:main Aug 25, 2025
89 checks passed
@trask trask deleted the support-custom-exception-handling branch October 20, 2025 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants