-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
feat(performance): Adds exclusive time and measurements to spans #10704
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
Changes from 2 commits
0639e97
f7d70a0
b990a55
2b495f1
7a019ea
0592b2b
3eff09f
1ed43e3
7156838
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
import type { TraceContext } from './context'; | ||
import type { Instrumenter } from './instrumenter'; | ||
import type { Measurements } from './measurement'; | ||
import type { Primitive } from './misc'; | ||
import type { HrTime } from './opentelemetry'; | ||
import type { Transaction } from './transaction'; | ||
|
@@ -21,13 +22,15 @@ export type SpanAttributeValue = | |
| boolean | ||
| Array<null | undefined | string> | ||
| Array<null | undefined | number> | ||
| Array<null | undefined | boolean>; | ||
| Array<null | undefined | boolean> | ||
| Measurements; | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This breaks OTEL compatibility unfortunately. We'll just add them to span data directly for now, but we need a bigger strategy afterwards. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah I think we'll probably have to flatten measurements into the attributes, or something similar in the future |
||
|
||
export type SpanAttributes = Partial<{ | ||
'sentry.origin': string; | ||
'sentry.op': string; | ||
'sentry.source': string; | ||
'sentry.sample_rate': number; | ||
measurements: Measurements; | ||
}> & | ||
Record<string, SpanAttributeValue | undefined>; | ||
|
||
|
@@ -178,6 +181,11 @@ export interface SpanContext { | |
* The origin of the span, giving context about what created the span. | ||
*/ | ||
origin?: SpanOrigin; | ||
|
||
/** | ||
* Exclusive time in milliseconds. | ||
*/ | ||
exclusiveTime?: number; | ||
} | ||
|
||
/** Span holding trace_id, span_id */ | ||
|
Uh oh!
There was an error while loading. Please reload this page.