Skip to content

Commit b8c14c6

Browse files
committed
Disable Appsignal monitoring when invalidating signatures
For the same reasons as #384 we should disable the Appsignal monitoring so that we don't use large amounts of memory when iterating over a job that invalidates hundreds of thousands or even millions of signatures.
1 parent 550258f commit b8c14c6

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

app/models/invalidation.rb

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -166,13 +166,15 @@ def invalidate!
166166
counted_at: Time.current
167167
)
168168

169-
matching_signatures.find_in_batches(batch_size: 100) do |signatures|
170-
signatures.each do |signature|
171-
signature.invalidate!(Time.current, self)
172-
increment!(:invalidated_count)
169+
Appsignal.without_instrumentation do
170+
matching_signatures.find_in_batches(batch_size: 100) do |signatures|
171+
signatures.each do |signature|
172+
signature.invalidate!(Time.current, self)
173+
increment!(:invalidated_count)
174+
end
175+
176+
reload and return if cancelled?
173177
end
174-
175-
reload and return if cancelled?
176178
end
177179

178180
update(completed_at: Time.current)

0 commit comments

Comments
 (0)