-
Notifications
You must be signed in to change notification settings - Fork 85
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(metrics): Extract measurement ratings, port from frontend [INGEST-542] #1130
Conversation
We need to tag those in metrics because we probably don't have sufficient querying capabilities to query for thresholds/histograms from metric distributions. "rating" is a made-up term, I couldn't find prior art for a term that describes "the thing coming from thresholds". Extracted from: https://github.com/getsentry/sentry/blob/bff64dcf2722dec969d0b9e82366855382238930/static/app/views/performance/landing/widgets/widgetDefinitions.tsx#L94-L133
relay-server/src/actors/envelopes.rs
Outdated
"measurement.fcp" => rate_range(1000.0, 3000.0), | ||
"measurement.fid" => rate_range(100.0, 300.0), | ||
"measurement.cls" => rate_range(0.1, 0.25), | ||
_ => "unknown".to_owned(), |
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.
Should we return None
here instead, and only insert the tag if Some
?
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.
i prefer this because then you know for which buckets this classification ran at all
nvm that's not very useful, I'll remove
* master: test(outcomes): Fix sort order in flaky test (#1135) feat(outcomes): Aggregate more outcomes (#1134) ref(outcomes): Fold processing vs non-processing into single actor (#1133) build: Update symbolic to support UE5 (#1132) feat(metrics): Extract measurement ratings, port from frontend (#1130) feat(metrics): Another statsd metric to measure bucket duplication (#1128) feat(outcomes): Emit outcomes as client reports (#1119) fix: Move changelog line to right version (#1129) fix(dangerjs): Do not suggest to add JIRA ticket to changelog (#1125) feat(metrics): Tag metrics by transaction name [INGEST-542] (#1126)
We need to tag those in metrics because we probably don't have
sufficient querying capabilities to query for thresholds/histograms from
metric distributions.
"rating" is a made-up term, I couldn't find prior art for a term that
describes "the thing coming from thresholds".
Extracted from: https://github.com/getsentry/sentry/blob/bff64dcf2722dec969d0b9e82366855382238930/static/app/views/performance/landing/widgets/widgetDefinitions.tsx#L94-L133