[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