Skip to content

Conversation

@kddubey
Copy link
Contributor

@kddubey kddubey commented Nov 10, 2025

in the near future, sounds like we can get rid of this RPC and have overwatch supply it. for now, we should make this line up w/ the constraints that the actual tools have

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 10, 2025
@kddubey kddubey marked this pull request as ready for review November 10, 2025 17:22
@kddubey kddubey requested a review from a team as a code owner November 10, 2025 17:22
Copy link
Member

@JoshFerge JoshFerge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

an optimization might be to combine more of these db queries together, but i think how you have it works (also not sure tables are on separate databases these days)

@kddubey kddubey merged commit ee244d0 into master Nov 10, 2025
65 checks passed
@kddubey kddubey deleted the kddubey/bug-prediction/require-code-mappings branch November 10, 2025 21:44
Jesse-Box pushed a commit that referenced this pull request Nov 12, 2025
…103030)

in the near future, sounds like we can get rid of this RPC and have
overwatch supply it. for now, we should make this line up w/ the
[constraints that the actual tools
have](https://github.com/getsentry/sentry/blob/d1e24794df851ed5e93d298c564ae18b76d04d41/src/sentry/seer/fetch_issues/utils.py#L86)
andrewshie-sentry pushed a commit that referenced this pull request Nov 13, 2025
…103030)

in the near future, sounds like we can get rid of this RPC and have
overwatch supply it. for now, we should make this line up w/ the
[constraints that the actual tools
have](https://github.com/getsentry/sentry/blob/d1e24794df851ed5e93d298c564ae18b76d04d41/src/sentry/seer/fetch_issues/utils.py#L86)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants