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(discover): Prevent calling Snuba with an empty list of projects #69577
feat(discover): Prevent calling Snuba with an empty list of projects #69577
Conversation
I messed up #69499 so I had to open this one. |
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.
🙏 thanks for tackling this
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #69577 +/- ##
==========================================
- Coverage 79.68% 79.67% -0.01%
==========================================
Files 6483 6483
Lines 287949 287909 -40
Branches 49642 49619 -23
==========================================
- Hits 229438 229385 -53
- Misses 58140 58153 +13
Partials 371 371
|
Suspect IssuesThis pull request was deployed and Sentry observed the following issues:
Did you find this useful? React with a 👍 or 👎 |
Do not include a where condition if
project_ids
is an empty list since it will return no data.This will cause Snuba to respond that
project_ids
is missing rather than pretend that everything is fine.In #69575 I attempt to save the call to Snuba but it requires a lot of test changes.