Skip to content

feat(producer): Rolling backpressure out to FTP instances again - #120681

Merged
lvthanh03 merged 1 commit into
masterfrom
tony/re-rollout-ftp-with-backpressure
Jul 27, 2026
Merged

feat(producer): Rolling backpressure out to FTP instances again#120681
lvthanh03 merged 1 commit into
masterfrom
tony/re-rollout-ftp-with-backpressure

Conversation

@lvthanh03

Copy link
Copy Markdown
Member

Rolls backpressure out to FutureTrackingProducer after the previous attempt in #120470 and after a bug fix was made: getsentry/taskbroker#766.

Adds an option to enable FTP backpressure, this will require producers to restart after the option changes to take effect. Updates existing FTP instances to use this.

@lvthanh03
lvthanh03 requested review from a team as code owners July 27, 2026 19:00
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 27, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Want reviews to match your repository better? Bugbot Learning can learn team-specific rules from PR activity. A team admin can enable Learning in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 8ae7cb9. Configure here.

Comment thread src/sentry/utils/arroyo_producer.py
@lvthanh03
lvthanh03 enabled auto-merge (squash) July 27, 2026 19:11
@lvthanh03
lvthanh03 disabled auto-merge July 27, 2026 19:13
@lvthanh03
lvthanh03 merged commit ff5fc58 into master Jul 27, 2026
89 checks passed
@lvthanh03
lvthanh03 deleted the tony/re-rollout-ftp-with-backpressure branch July 27, 2026 19:24
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.

2 participants