Skip to content

Commit a8bf305

Browse files
committed
test: speed up rebalancer/receiving_bucket.test.lua
It makes lots of DML in the beginning which could be done in much fewer transactions. On an old MacBook this patch makes the test 7 seconds faster. NO_DOC=test
1 parent 3a4c80e commit a8bf305

File tree

2 files changed

+14
-4
lines changed

2 files changed

+14
-4
lines changed

test/rebalancer/receiving_bucket.result

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,16 @@ create_simple_space('test4')
7676
create_simple_space('test5', {engine = 'vinyl'})
7777
---
7878
...
79-
for i = 1, 10000 do box.space.test:replace{i, 1, 1} box.space.test2:replace{i, 1, 2} box.space.test3:replace{i, 1, 3} end
79+
box.begin() for i = 1, 10000 do box.space.test:replace{i, 1, 1} box.space.test2:replace{i, 1, 2} end box.commit()
8080
---
8181
...
82-
for i = 1, 500 do box.space.test4:replace{i, 1, 4} box.space.test5:replace{i, 1, 5} end
82+
box.begin() for i = 1, 10000 do box.space.test3:replace{i, 1, 3} end box.commit()
83+
---
84+
...
85+
box.begin() for i = 1, 500 do box.space.test4:replace{i, 1, 4} end box.commit()
86+
---
87+
...
88+
box.begin() for i = 1, 500 do box.space.test5:replace{i, 1, 5} end box.commit()
8389
---
8490
...
8591
box.snapshot()

test/rebalancer/receiving_bucket.test.lua

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,8 +33,12 @@ create_simple_space('test3', {engine = 'vinyl'})
3333
create_simple_space('test4')
3434
create_simple_space('test5', {engine = 'vinyl'})
3535

36-
for i = 1, 10000 do box.space.test:replace{i, 1, 1} box.space.test2:replace{i, 1, 2} box.space.test3:replace{i, 1, 3} end
37-
for i = 1, 500 do box.space.test4:replace{i, 1, 4} box.space.test5:replace{i, 1, 5} end
36+
box.begin() for i = 1, 10000 do box.space.test:replace{i, 1, 1} box.space.test2:replace{i, 1, 2} end box.commit()
37+
box.begin() for i = 1, 10000 do box.space.test3:replace{i, 1, 3} end box.commit()
38+
39+
box.begin() for i = 1, 500 do box.space.test4:replace{i, 1, 4} end box.commit()
40+
box.begin() for i = 1, 500 do box.space.test5:replace{i, 1, 5} end box.commit()
41+
3842
box.snapshot()
3943
box.space.test:count()
4044
box.space.test2:count()

0 commit comments

Comments
 (0)