-
-
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(spans): Tag perf problems created from standalone spans #66325
feat(spans): Tag perf problems created from standalone spans #66325
Conversation
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 although we probably want the span group-types to just fork the performance group types to keep things simple in the future.
@@ -85,7 +85,7 @@ def repeating_span(): | |||
|
|||
assert ( | |||
job["performance_problems"][0].fingerprint | |||
== "1-GroupType.PERFORMANCE_N_PLUS_ONE_DB_QUERIES-f906d576ffde8f005fd741f7b9c8a35062361e67" | |||
== "1-GroupType.PERFORMANCE_N_PLUS_ONE_DB_QUERIES-f906d576ffde8f005fd741f7b9c8a35062361e67-1019" |
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.
lol @ GroupType.PERFORMANCE_N_PLUS_ONE_DB_QUERIES
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #66325 +/- ##
==========================================
+ Coverage 84.24% 84.29% +0.04%
==========================================
Files 5294 5309 +15
Lines 236582 237228 +646
Branches 40962 41034 +72
==========================================
+ Hits 199317 199965 +648
+ Misses 37046 37044 -2
Partials 219 219
|
This is just temporary as we test the pipeline, I want to collect some metrics so we are reasonably confident we're not introducing a ton of new fingerprints and catching more or less the same issues |
As we set up the application logic to run performance issue detection on standalone spans, we want to collect metrics to check if transaction based and span based perf issue detection are equivalent. Tagging metrics with `is_standalone_spans` to track this. This also updates fingerprint so the standalone span occurrences don't get grouped with existing perf issues during testing.
As we set up the application logic to run performance issue detection
on standalone spans, we want to collect metrics to check if transaction
based and span based perf issue detection are equivalent. Tagging
metrics with
is_standalone_spans
to track this.This also updates fingerprint so the standalone span occurrences
don't get grouped with existing perf issues during testing.