Skip to content

Commit d14a0f9

Browse files
authored
ref(trends): Impose strict top event limit (#49098)
This is a temporary fix to select top 8 events. For each transaction name we're getting two timeseries (trend_function and count) of 1 hour bucket for 14 days (the default stats period) which amounts to 1*24*14*2=672 data points. I'm picking 8 events very conservatively here until I can either paginate the query or parallelize it.
1 parent 0c9ab30 commit d14a0f9

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

src/sentry/api/endpoints/organization_events_trendsv2.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@ def get(self, request: Request, organization) -> Response:
7575
selected_columns.append(trend_function)
7676
selected_columns.append("count()")
7777
request.yAxis = selected_columns
78+
top_events_limit = 8
7879

7980
def get_top_events(selected_columns, user_query, params, orderby, limit, referrer):
8081
return query(
@@ -102,7 +103,7 @@ def get_event_stats_metrics(_, user_query, params, rollup, zerofill_results, __)
102103
user_query=user_query,
103104
params=params,
104105
orderby=["-count()"],
105-
limit=100,
106+
limit=top_events_limit,
106107
referrer=Referrer.API_TRENDS_GET_EVENT_STATS_V2_TOP_EVENTS.value,
107108
)
108109

@@ -167,7 +168,7 @@ def get_event_stats_metrics(_, user_query, params, rollup, zerofill_results, __)
167168
request,
168169
organization,
169170
get_event_stats_metrics,
170-
top_events=50,
171+
top_events=top_events_limit,
171172
query_column=trend_function,
172173
params=params,
173174
query=_query,

0 commit comments

Comments
 (0)