Skip to content

ref(replay): bump max segment limit to 150 for replay summaries #103274

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 13, 2025
Merged

ref(replay): bump max segment limit to 150 for replay summaries #103274

merged 1 commit into from
Nov 13, 2025

Conversation

@michellewzhang
Copy link
Member

@michellewzhang michellewzhang commented Nov 12, 2025 •
edited
Loading

bump up to 150. we haven't seen an overwhelming instance of unique web replays hitting 100+ segments in the past 30 days so it should be safe.

frontend pr to follow

@michellewzhang michellewzhang requested a review from a team as a code owner November 12, 2025 23:20
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 12, 2025
@michellewzhang michellewzhang requested review from a team and billyvg November 12, 2025 23:21
@michellewzhang michellewzhang mentioned this pull request Nov 12, 2025
Merged
@billyvg
Copy link
Member

billyvg commented Nov 13, 2025

Let's keep an eye on https://sentry.sentry.io/explore/traces/?aggregateField=%7B%22groupBy%22%3A%22%22%7D&aggregateField=%7B%22yAxes%22%3A%5B%22p95%28span.duration%29%22%5D%7D&field=id&field=span.op&field=span.description&field=span.duration&field=transaction&field=timestamp&field=span.name&mode=aggregate&project=1&project=6178942&query=transaction%3Aseer.automation.summarize.replays.start_replay_breadcrumbs_summary&statsPeriod=30d after release

michellewzhang added a commit that referenced this pull request Nov 13, 2025
@michellewzhang
ref(replay): update segment count on frontend (#103275)
60424d5 
follow up pr to #103274
@michellewzhang michellewzhang merged commit 601dd94 into master Nov 13, 2025
68 checks passed
@michellewzhang michellewzhang deleted the mz/add-num-seg-to-log branch November 13, 2025 17:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@aliu39 aliu39 aliu39 approved these changes

@billyvg billyvg Awaiting requested review from billyvg

Assignees

No one assigned

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@michellewzhang @billyvg @aliu39