Fix transaction pollution in Shard.by_name

5c45c21d · Nick Thomas · 9ed1c2b5 · 5c45c21d
Commit 5c45c21d authored Dec 07, 2018 by Nick Thomas
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 1 deletion

app/models/shard.rb app/models/shard.rb +3 -1

No files found.
--- a/app/models/shard.rb
+++ b/app/models/shard.rb
@@ -18,7 +18,9 @@ class Shard < ActiveRecord::Base
  end
  def self.by_name(name)
-    find_or_create_by(name: name)
+    transaction(requires_new: true) do
+      find_or_create_by(name: name)
+    end
  rescue ActiveRecord::RecordNotUnique
    retry
  end