Skip to content

ref(api): remove OrganizationEventsSpansHistogramEndpoint #110339

Merged
cvxluo merged 2 commits intomasterfrom
cvxluo/lqnoulxtrlou
Mar 11, 2026
Merged

ref(api): remove OrganizationEventsSpansHistogramEndpoint #110339
cvxluo merged 2 commits intomasterfrom
cvxluo/lqnoulxtrlou

Conversation

@cvxluo
Copy link
Contributor

@cvxluo cvxluo commented Mar 10, 2026

The frontend consumer (spanHistogramQuery.tsx) was deleted in #99522 (Sep 2025) when the transaction spans page was removed. The endpoint has had zero callers since.

Removing spans_histogram_query in a followup

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 10, 2026
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Mar 10, 2026
@github-actions
Copy link
Contributor

🚨 Warning: This pull request contains Frontend and Backend changes!

It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently.

Have questions? Please ask in the #discuss-dev-infra channel.

The frontend consumer (spanHistogramQuery.tsx) was deleted in #99522
(Sep 2025) when the transaction spans page was removed. The endpoint
has had zero callers since. It was PRIVATE and never publicly documented.

Co-authored-by: Claude <noreply@anthropic.com>

chore(api): remove spans histogram from allowlists

Co-authored-by: Claude <noreply@anthropic.com>
@cvxluo cvxluo marked this pull request as ready for review March 11, 2026 00:13
@cvxluo cvxluo requested review from a team as code owners March 11, 2026 00:13
@cvxluo cvxluo requested a review from a team March 11, 2026 00:13
Copy link
Member

@wedamija wedamija left a comment

Choose a reason for hiding this comment

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

🔥

@cvxluo cvxluo merged commit e8d2f3f into master Mar 11, 2026
58 checks passed
@cvxluo cvxluo deleted the cvxluo/lqnoulxtrlou branch March 11, 2026 16:55
cvxluo added a commit that referenced this pull request Mar 11, 2026
Follow up of #110339. These
functions in the Snuba discover module were only called by the
now-removed OrganizationEventsSpansHistogramEndpoint. Also removes the
`spans_histogram` and `fn_span_exclusive_time` SnQL function definitions
and associated referrer constants.

Co-authored-by: Claude <noreply@anthropic.com>
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 Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants