-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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(performance) Add related issues endpoint #18344
Conversation
A first pass at a related issues endpoint to be used by the transaction summary page to show issues related to that transaction.
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.
Some minor stuff, but I think this is a good first pass to try related issues
from sentry.snuba import discover | ||
|
||
|
||
IssueSeach = EventsDatasetSnubaSearchBackend(**settings.SENTRY_SEARCH_OPTIONS) |
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.
Generally we don't build specific backends, and use the search proxy instead.
from sentry import search
results = search.query(...)
I saw that this is also being done the way you have it in OrganizationGroupIndex
. Perhaps @iProgramStuff knows why we're using a specific backend there?
A first pass at a related issues endpoint to be used by the transaction summary
page to show issues related to that transaction.