Skip to content

Commit

Permalink
concurrency: fix test cases
Browse files Browse the repository at this point in the history
There were a few tests where non-transactional requests were trying to
acquire exclusive locks; this isn't allowed -- non-transactional
requests can only perform writes by laying intents. We fix these tests
by using the correct lock strength in such cases (lock.Intent).

Epic: none

Release note: None
  • Loading branch information
arulajmani committed Aug 2, 2023
1 parent 9355e10 commit 6805624
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -722,7 +722,7 @@ num=0
# waiters.
# -----------------------------------------------------------------------------

new-request r=req11 txn=none ts=12,1 spans=exclusive@a
new-request r=req11 txn=none ts=12,1 spans=intent@a
----

scan r=req11
Expand Down Expand Up @@ -825,12 +825,12 @@ new-request r=req13 txn=txn6 ts=11,0 spans=exclusive@a+exclusive@b+exclusive@c+e
new-request r=req14 txn=txn5 ts=11,0 spans=exclusive@a+exclusive@b
----

# Non-locking, transactional request
# Non-locking, transactional request.
new-request r=req15 txn=txn5 ts=11,0 spans=none@c+none@d
----

# Non-locking, transactional request
new-request r=req16 txn=none ts=11,0 spans=exclusive@e+exclusive@f
# Non-transactional write request.
new-request r=req16 txn=none ts=11,0 spans=intent@e+intent@f
----

acquire r=req13 k=a durability=u strength=exclusive
Expand Down

0 comments on commit 6805624

Please sign in to comment.