-
Notifications
You must be signed in to change notification settings - Fork 1
Expand file tree
/
Copy path06_read_skew_spec.rb
More file actions
113 lines (96 loc) · 3.91 KB
/
Copy path06_read_skew_spec.rb
File metadata and controls
113 lines (96 loc) · 3.91 KB
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
RSpec.describe 'Read skew versus repeatable read isolation level' do
around do |example|
execute <<~SQL
CREATE TABLE events (
id text NOT NULL,
available_seats integer NOT NULL CHECK (available_seats >= 0),
PRIMARY KEY (id)
);
SQL
execute <<~SQL
CREATE TABLE bookings (
id uuid DEFAULT gen_random_uuid() NOT NULL,
customer_name text NOT NULL,
seat_count integer NOT NULL,
event_id text NOT NULL,
FOREIGN KEY (event_id) REFERENCES events (id),
PRIMARY KEY (id)
);
SQL
example.run
ensure
execute 'DROP TABLE IF EXISTS bookings;'
execute 'DROP TABLE IF EXISTS events;'
end
before do
transaction do
Event.create!(id: 'event_a', available_seats: 3)
Booking.create!(customer_name: 'Alice', seat_count: 1, event_id: 'event_a')
Booking.create!(customer_name: 'Bob', seat_count: 2, event_id: 'event_a')
Event.create!(id: 'event_b', available_seats: 4)
Booking.create!(customer_name: 'Alice', seat_count: 1, event_id: 'event_b')
Booking.create!(customer_name: 'Bob', seat_count: 2, event_id: 'event_b')
end
end
let(:alice) do
define('alice') do
transaction(isolation: :repeatable_read) do
Booking.where(customer_name: 'Bob', event_id: 'event_b').update_all('seat_count = seat_count - 1')
Event.increment_counter(:available_seats, 'event_b', by: 1)
synchronizer[:alice_update_started] = true
wait_for(seconds: 2) # Wait for Bob's inner SELECT statement to finish and UPDATE to start waiting
end
end
end
let(:bob) do
define('bob') do
transaction(isolation: :repeatable_read) do
wait_until { synchronizer[:alice_update_started] }
updated_bookings = log execute(<<~SQL.lstrip).to_a
UPDATE bookings SET seat_count = seat_count - 1
WHERE bookings.id IN (
SELECT bookings.id
FROM bookings
WHERE bookings.customer_name = 'Bob'
AND bookings.event_id IN (
SELECT bookings.event_id
FROM bookings
WHERE bookings.customer_name IN ('Bob', 'Alice')
GROUP BY bookings.event_id
HAVING (SUM(seat_count) > 2)
)
)
RETURNING id, event_id
SQL
updated_bookings.each do |booking|
Event.increment_counter(:available_seats, booking['event_id'], by: 1)
end
end
end
end
specify <<-DESC.lstrip do
Read skew anomaly is avoided;
serialization failure is raised for Bob since Alice commits her transaction first;
on retry, Bob sees that Alice has already returned one of his seats and avoids
accidentally returning his other seat;
all reads in a repeatable read transaction see the same snapshot of the database;
the UPDATE sees another snapshot since it has to re-fetch the latest version of the bookings;
a repeatable read transaction detects this and raises a serialization failure instead of allowing
the update to proceed
DESC
event_b_initially_available_seats = log Event.where(id: 'event_b').pluck(:available_seats).first
expect(event_b_initially_available_seats).to eq(4)
start_in_order_and_conduct(
[alice, { execute_without_coordination: true }],
[bob, { execute_without_coordination: true, retry_on: [ActiveRecord::SerializationFailure] }]
)
expect(outcomes(bob, alice)).to match_array(%i[success success])
# Only one of Bob's seats is returned by Alice.
# Bob retries his transaction after encountering a serialization failure
# and sees that Alice has already returned only one of his seats.
event_b_available_seats = log Event.where(id: 'event_b').pluck(:available_seats).first
expect(event_b_available_seats).to eq(5)
event_b_taken_seats = log Booking.where(event_id: 'event_b').sum(:seat_count)
expect(event_b_taken_seats).to eq(2)
end
end