Skip to content

ref: Migrate webhook delivery ThreadPoolExecutor#111465

Merged
gricha merged 1 commit intomasterfrom
gricha/ref/migrate-webhook-delivery-threadpool
Mar 25, 2026
Merged

ref: Migrate webhook delivery ThreadPoolExecutor#111465
gricha merged 1 commit intomasterfrom
gricha/ref/migrate-webhook-delivery-threadpool

Conversation

@gricha
Copy link
Member

@gricha gricha commented Mar 24, 2026

Migrate deliver_webhooks.py to ContextPropagatingThreadPoolExecutor.

Split out from the main migration PR (#111460) for separate review — this is a high-throughput webhook delivery path that uses ThreadPoolExecutor with as_completed to fan out webhook deliveries across cells. Context propagation here ensures Sentry SDK tracing spans are properly linked across the thread pool workers.

Depends on #111460

Agent transcript: https://claudescope.sentry.dev/share/qPBlInQUSChZ__7X4OQMJ71LrObNmN7u3erNuSNh-yI

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Mar 24, 2026
@gricha gricha marked this pull request as ready for review March 24, 2026 21:34
@gricha gricha requested a review from a team as a code owner March 24, 2026 21:34
Base automatically changed from gricha/ref/migrate-threadpool-to-context-propagating to master March 24, 2026 22:17
@gricha gricha requested review from a team as code owners March 24, 2026 22:17
Migrate deliver_webhooks.py to ContextPropagatingThreadPoolExecutor.
Split out from the main migration PR for separate review since this
is a high-throughput webhook delivery path.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Agent transcript: https://claudescope.sentry.dev/share/sWUe-etgvVnFo9gJJmgW5sUXFMi_86dibqPPO7z0N8c
@gricha gricha force-pushed the gricha/ref/migrate-webhook-delivery-threadpool branch from a534195 to a97a12a Compare March 24, 2026 22:30
@gricha gricha merged commit 70c99f2 into master Mar 25, 2026
64 checks passed
@gricha gricha deleted the gricha/ref/migrate-webhook-delivery-threadpool branch March 25, 2026 01:16
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