diff options
author | Quan Chau <quan.chau@appfolio.com> | 2018-12-27 10:59:18 -0800 |
---|---|---|
committer | Quan Chau <quan.chau@appfolio.com> | 2019-04-08 12:02:41 -0700 |
commit | 66762b81f45a4499a749378bbf14eb64bc097d3a (patch) | |
tree | a1bb1c0ba5b35d43c2777677afc0e558f62918d2 /activerecord | |
parent | 95a1d8721c84d13951f0a531e0dda6233d3180d4 (diff) | |
download | rails-66762b81f45a4499a749378bbf14eb64bc097d3a.tar.gz rails-66762b81f45a4499a749378bbf14eb64bc097d3a.tar.bz2 rails-66762b81f45a4499a749378bbf14eb64bc097d3a.zip |
Wrap Mysql count of deleted rows in lock block to avoid conflict in test
Diffstat (limited to 'activerecord')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/mysql/database_statements.rb | 4 | ||||
-rw-r--r-- | activerecord/test/cases/adapters/mysql2/count_deleted_rows_with_lock_test.rb | 28 |
2 files changed, 31 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/mysql/database_statements.rb b/activerecord/lib/active_record/connection_adapters/mysql/database_statements.rb index a14394fe04..2132e5d248 100644 --- a/activerecord/lib/active_record/connection_adapters/mysql/database_statements.rb +++ b/activerecord/lib/active_record/connection_adapters/mysql/database_statements.rb @@ -61,7 +61,9 @@ module ActiveRecord def exec_delete(sql, name = nil, binds = []) if without_prepared_statement?(binds) - execute_and_free(sql, name) { @connection.affected_rows } + @lock.synchronize do + execute_and_free(sql, name) { @connection.affected_rows } + end else exec_stmt_and_free(sql, name, binds) { |stmt| stmt.affected_rows } end diff --git a/activerecord/test/cases/adapters/mysql2/count_deleted_rows_with_lock_test.rb b/activerecord/test/cases/adapters/mysql2/count_deleted_rows_with_lock_test.rb new file mode 100644 index 0000000000..4d361e405c --- /dev/null +++ b/activerecord/test/cases/adapters/mysql2/count_deleted_rows_with_lock_test.rb @@ -0,0 +1,28 @@ +# frozen_string_literal: true + +require "cases/helper" +require "support/connection_helper" +require "models/author" +require "models/bulb" + +module ActiveRecord + class CountDeletedRowsWithLockTest < ActiveRecord::Mysql2TestCase + test "delete and create in different threads synchronize correctly" do + Bulb.unscoped.delete_all + Bulb.create!(name: "Jimmy", color: "blue") + + delete_thread = Thread.new do + Bulb.unscoped.delete_all + end + + create_thread = Thread.new do + Author.create!(name: "Tommy") + end + + delete_thread.join + create_thread.join + + assert_equal 1, delete_thread.value + end + end +end |