-
-
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
fix(release-alert): Update Query to use Org & Project Filtering for Release Health #71386
fix(release-alert): Update Query to use Org & Project Filtering for Release Health #71386
Conversation
UseCaseID.SESSIONS, org_id, SessionMRI.RAW_SESSION.value | ||
), | ||
), | ||
Condition(Column("org_id"), Op.GTE, prev_org_id), |
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.
We might not have processed all the projects for an org.
06b402f
to
20aee36
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #71386 +/- ##
=======================================
Coverage 77.87% 77.87%
=======================================
Files 6538 6538
Lines 291235 291277 +42
Branches 50391 50400 +9
=======================================
+ Hits 226809 226845 +36
Misses 58187 58187
- Partials 6239 6245 +6
|
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
@wedamija if we don't want/need to do this, i can close this PR |
Helps mitigate https://getsentry.atlassian.net/browse/ISSUE-1702
I create an option to query switch between the old query and the new query.
The new query is the same as old query p much, but with few differences. we sort by
org_id
andproject_id
and use the last one as the filter for the next query.