Skip to content

Commit e20bab8

Browse files
solnicCodex
andcommitted
refactor(sidekiq): drop redundant span clearing
Co-Authored-By: Codex <noreply@example.com>
1 parent dab09c6 commit e20bab8

2 files changed

Lines changed: 0 additions & 5 deletions

File tree

‎sentry-ruby/lib/sentry/scope.rb‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -218,10 +218,6 @@ def set_span(span)
218218
@span = span
219219
end
220220

221-
def clear_span
222-
@span = nil
223-
end
224-
225221
# @!macro set_user
226222
def set_user(user_hash)
227223
check_argument_type!(user_hash, Hash)

‎sentry-sidekiq/lib/sentry/sidekiq/sentry_context_middleware.rb‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@ def perform_through_active_job(job, sidekiq_job)
5353
Sentry.clone_hub_to_current_thread
5454

5555
Sentry.with_scope do |scope|
56-
scope.clear_span
5756
consumer.apply_scope_data(scope, sidekiq_job, default_scope_data: job_extension.scope_data(job: sidekiq_job))
5857
scope.set_transaction_name(job_extension.transaction_name(job: job), source: :task)
5958
trace_headers = args.first.is_a?(Hash) && args.first.dig("_sentry", "trace_propagation_headers")

0 commit comments

Comments
 (0)