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
POTEL 2 - Promote OpenTelemetry Span attributes #3402
base: feat/potel-1-context-forking-and-basics
Are you sure you want to change the base?
POTEL 2 - Promote OpenTelemetry Span attributes #3402
Conversation
Instructions and example for changelogPlease add an entry to Example: ## Unreleased
- POTEL 2 - Promote OpenTelemetry Span attributes ([#3402](https://github.com/getsentry/sentry-java/pull/3402)) If none of the above apply, you can opt out of this check by adding |
Performance metrics 🚀
|
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9931715 | 374.85 ms | 431.48 ms | 56.62 ms |
9a9af11 | 376.72 ms | 442.33 ms | 65.61 ms |
f5f9136 | 395.57 ms | 504.94 ms | 109.36 ms |
App size
Revision | Plain | With Sentry | Diff |
---|---|---|---|
9931715 | 1.70 MiB | 2.28 MiB | 596.01 KiB |
9a9af11 | 1.70 MiB | 2.28 MiB | 596.17 KiB |
f5f9136 | 1.70 MiB | 2.28 MiB | 596.01 KiB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, left suggestions for deduplicating some code
} | ||
|
||
@SuppressWarnings("deprecation") | ||
private OtelSpanInfo extractSpanDescription(SpanData otelSpan) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't we have the overloaded method receiving ReadableSpan
call this method with ReadableSpan.toSpanData()
?
} | ||
|
||
@SuppressWarnings("deprecation") | ||
private OtelSpanInfo descriptionForHttpMethod( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't we have the overloaded method receiving ReadableSpan
call this method with ReadableSpan.toSpanData()
?
} | ||
|
||
@SuppressWarnings("deprecation") | ||
private OtelSpanInfo descriptionForDbSystem(final @NotNull SpanData otelSpan) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couldn't we have the overloaded method receiving ReadableSpan
call this method with ReadableSpan.toSpanData()
?
…l-2-promote-span-attributes
…l-2-promote-span-attributes
📜 Description
💡 Motivation and Context
Sentry features like "Requests" need certain attributes to work, e.g. span
op
should be "METHOD URL", response status code should be in span data.💚 How did you test it?
📝 Checklist
sendDefaultPII
is enabled.🔮 Next steps