No need for this to be in a transaction
If you update the name by itself, it is fine. We generally want to be very careful about having txn spans, since all the work done within them will block the sqlite writer.
This commit is contained in:
@@ -20,10 +20,8 @@ class BucketsController < ApplicationController
|
||||
end
|
||||
|
||||
def update
|
||||
@bucket.transaction do
|
||||
@bucket.update! bucket_params
|
||||
@bucket.accesses.revise granted: grantees, revoked: revokees
|
||||
end
|
||||
@bucket.update! bucket_params
|
||||
@bucket.accesses.revise granted: grantees, revoked: revokees
|
||||
|
||||
redirect_to bucket_bubbles_path(@bucket)
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user