Skip to content

distinguish between shared prewrite locks and shared pessimistic locks#1377

Merged
cfzjywxk merged 3 commits intopingcap:tidb-8.5-with-shared-lockfrom
zyguan:dev/shared-lock
Nov 17, 2025
Merged

distinguish between shared prewrite locks and shared pessimistic locks#1377
cfzjywxk merged 3 commits intopingcap:tidb-8.5-with-shared-lockfrom
zyguan:dev/shared-lock

Conversation

@zyguan
Copy link
Contributor

@zyguan zyguan commented Nov 14, 2025

Signed-off-by: zyguan <zhongyangguan@gmail.com>
Signed-off-by: zyguan <zhongyangguan@gmail.com>
Signed-off-by: zyguan <zhongyangguan@gmail.com>
@zyguan zyguan changed the title let LockInfo carry the full txn info when it's shared distinguish between shared prewrite locks and shared pessimistic locks Nov 17, 2025
@ti-chi-bot
Copy link

ti-chi-bot bot commented Nov 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfzjywxk, you06

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot
Copy link

ti-chi-bot bot commented Nov 17, 2025

[LGTM Timeline notifier]

Timeline:

  • 2025-11-17 07:22:05.995045274 +0000 UTC m=+1291575.438075153: ☑️ agreed by you06.
  • 2025-11-17 07:34:06.410354123 +0000 UTC m=+1292295.853384012: ☑️ agreed by cfzjywxk.

@cfzjywxk cfzjywxk merged commit 2e49090 into pingcap:tidb-8.5-with-shared-lock Nov 17, 2025
1 of 2 checks passed
you06 pushed a commit to you06/kvproto that referenced this pull request Nov 17, 2025
pingcap#1377)

* let LockInfo carry the full txn info when it's shared

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* move the todo comment to the proper place

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks

Signed-off-by: zyguan <zhongyangguan@gmail.com>

---------

Signed-off-by: zyguan <zhongyangguan@gmail.com>
you06 pushed a commit to you06/kvproto that referenced this pull request Dec 16, 2025
pingcap#1377)

* let LockInfo carry the full txn info when it's shared

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* move the todo comment to the proper place

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks

Signed-off-by: zyguan <zhongyangguan@gmail.com>

---------

Signed-off-by: zyguan <zhongyangguan@gmail.com>
ti-chi-bot bot pushed a commit that referenced this pull request Dec 18, 2025
…ared lock in pessimistic transacitons. (#1374)

* reserve Op for shared lock

Signed-off-by: you06 <you1474600@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks (#1377)

* let LockInfo carry the full txn info when it's shared

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* move the todo comment to the proper place

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks

Signed-off-by: zyguan <zhongyangguan@gmail.com>

---------

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* update kvproto

Signed-off-by: you06 <you1474600@gmail.com>

* add embed shared_lock_infos

Signed-off-by: you06 <you1474600@gmail.com>

---------

Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: zyguan <zhongyangguan@gmail.com>
Co-authored-by: zyguan <zhongyangguan@gmail.com>
you06 added a commit to you06/kvproto that referenced this pull request Feb 13, 2026
…ared lock in pessimistic transacitons. (pingcap#1374)

* reserve Op for shared lock

Signed-off-by: you06 <you1474600@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks (pingcap#1377)

* let LockInfo carry the full txn info when it's shared

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* move the todo comment to the proper place

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* distinguish between shared prewrite locks and shared pessimistic locks

Signed-off-by: zyguan <zhongyangguan@gmail.com>

---------

Signed-off-by: zyguan <zhongyangguan@gmail.com>

* update kvproto

Signed-off-by: you06 <you1474600@gmail.com>

* add embed shared_lock_infos

Signed-off-by: you06 <you1474600@gmail.com>

---------

Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: zyguan <zhongyangguan@gmail.com>
Co-authored-by: zyguan <zhongyangguan@gmail.com>
ti-chi-bot bot pushed a commit that referenced this pull request Feb 24, 2026
…ared lock in pessimistic transacitons. (#1374) (#1417)

* reserve Op for shared lock



* distinguish between shared prewrite locks and shared pessimistic locks (#1377)

* let LockInfo carry the full txn info when it's shared



* move the todo comment to the proper place



* distinguish between shared prewrite locks and shared pessimistic locks



---------



* update kvproto



* add embed shared_lock_infos



---------

Signed-off-by: you06 <you1474600@gmail.com>
Signed-off-by: zyguan <zhongyangguan@gmail.com>
Co-authored-by: zyguan <zhongyangguan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants