-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
feat(new-trace): add cache metrics to transaction trace details #70868
Conversation
@@ -49,7 +49,7 @@ export const DataTitles: Record<DataKey, string> = { | |||
'avg(cache.item_size)': t('Avg Value Size'), | |||
unsuccessfulHTTPCodes: t('Response Codes (3XX, 4XX, 5XX)'), | |||
httpCodeBreakdown: t('Response Code Breakdown'), | |||
cacheMissRate: t('Miss Rate'), | |||
'cache_miss_rate()': t('Miss Rate'), |
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.
Threw in an extra refactor here, sorry about that
Bundle ReportChanges will increase total bundle size by 7.96kB ⬆️
|
static/app/views/performance/newTraceDetails/traceDrawer/details/transaction/index.tsx
Outdated
Show resolved
Hide resolved
static/app/views/performance/newTraceDetails/traceDrawer/details/transaction/index.tsx
Outdated
Show resolved
Hide resolved
@JonasBa I've addressed you're comments about allowing cache metrics to load independently and mapping all the cache metrics instead of the first element. Lmk what u think! |
@DominikB2014 I'm really sorry about this, I must have missed the gh notification. Please feel free to ping me sooner next time, I try to reply to all notifications in less than a day |
Adds cache metrics section to a transaction node, if the transaction has cache metrics <img width="393" alt="image" src="https://github.com/getsentry/sentry/assets/44422760/d1bdf928-b665-4446-ba8e-e188a62462f2">
Adds cache metrics section to a transaction node, if the transaction has cache metrics