[ARVADOS] created: 505b7680f8ed8f518f1c3368eb4107e493115b23
Git user
git at public.curoverse.com
Wed Mar 30 15:00:31 EDT 2016
at 505b7680f8ed8f518f1c3368eb4107e493115b23 (commit)
commit 505b7680f8ed8f518f1c3368eb4107e493115b23
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date: Wed Mar 30 15:00:10 2016 -0400
8840: Use 'with_lock' instead of 'transaction' in Job.lock method.
diff --git a/services/api/app/models/job.rb b/services/api/app/models/job.rb
index afaae26..0115407 100644
--- a/services/api/app/models/job.rb
+++ b/services/api/app/models/job.rb
@@ -93,8 +93,7 @@ class Job < ArvadosModel
end
def lock locked_by_uuid
- transaction do
- self.reload
+ with_lock do
unless self.state == Queued and self.is_locked_by_uuid.nil?
raise AlreadyLockedError
end
-----------------------------------------------------------------------
hooks/post-receive
--
More information about the arvados-commits
mailing list