-
-
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
ref(replays): refactor query code #69334
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #69334 +/- ##
==========================================
+ Coverage 78.66% 79.68% +1.02%
==========================================
Files 6622 6446 -176
Lines 294737 286426 -8311
Branches 49377 49350 -27
==========================================
- Hits 231855 228241 -3614
+ Misses 62460 57748 -4712
- Partials 422 437 +15
|
…response (filled in by VALID_FIELD_SET)
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 🥀 |
Closing as this is outdated and a lot of changes to review - might come back to later |
query_replays_collection_raw
toquery_replays_collection_paginated
, and removequery_replays_collection
(this is no longer needed for compatibility with getsentry, since we moved the delete replays script to sentry)post_process.generate_normalized_output
and group/alphabetize the fields