Skip to content

Commit babad40

Browse files
authored
Merge pull request rails#51000 from Shopify/fix-race-condition
Fix a race condition in FutureResult#instrument
2 parents c13b82a + f7eb822 commit babad40

File tree

1 file changed

+5
-2
lines changed

1 file changed

+5
-2
lines changed

activerecord/lib/active_record/future_result.rb

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,11 @@ def initialize(future_result, instrumenter)
3232

3333
def instrument(name, payload = {}, &block)
3434
event = @instrumenter.new_event(name, payload)
35-
@events << event
36-
event.record(&block)
35+
begin
36+
event.record(&block)
37+
ensure
38+
@events << event
39+
end
3740
end
3841

3942
def flush

0 commit comments

Comments
 (0)