Skip to content

chore(huey): drop transaction based tracing - #7243

Draft
pnagre05 wants to merge 2 commits into
getsentry:major/3.0from
pnagre05:pnagre/major/remove-transactions-huey
Draft

chore(huey): drop transaction based tracing#7243
pnagre05 wants to merge 2 commits into
getsentry:major/3.0from
pnagre05:pnagre/major/remove-transactions-huey

Conversation

@pnagre05

@pnagre05 pnagre05 commented Aug 24, 2026

Copy link
Copy Markdown

Description

I removed the transaction-based tracing fallback from the Huey integration - I did this by doing the following:

  • Removed all has_span_streaming_enabled branching and enforced the use of sentry_sdk.traces.start_span for both patch_enqueue and patch_execute
  • Removed legacy compatibility utilities like the else: fallback blocks and the legacy continue_trace/set_status calls on scope.transaction
  • Deleted transaction-based tests (like the has_span_streaming parameterization) and updated the remaining test suite to unconditionally validate span streaming payloads instead of legacy transactions

Issues

Reminders

@pnagre05
pnagre05 requested a review from a team as a code owner August 24, 2026 18:50
@pnagre05
pnagre05 marked this pull request as draft August 24, 2026 18:50

@sentrivana sentrivana 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.

Hey @pnagre05, thanks for this one, too. It looks great, just two minor comments. I'll also run CI on it.

Thanks especially for migrating the tests that we apparently had forgotten about when adding span streaming to this integration, like test_huey_enqueue.

Comment thread sentry_sdk/integrations/huey.py Outdated
Comment thread tests/integrations/huey/test_huey.py Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants