-
Notifications
You must be signed in to change notification settings - Fork 5.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
*: try use max uint64 as tso for table/index reader which only has 1 cop task and in autocommit txn #56016
base: master
Are you sure you want to change the base?
*: try use max uint64 as tso for table/index reader which only has 1 cop task and in autocommit txn #56016
Conversation
Signed-off-by: crazycs520 <crazycs520@gmail.com>
…eader-try-use-max-uint64-as-tso
Signed-off-by: crazycs520 <crazycs520@gmail.com>
…eader-try-use-max-uint64-as-tso
Hi @crazycs520. Thanks for your PR. PRs from untrusted users cannot be marked as trusted with I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
/retest-required |
@crazycs520: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #56016 +/- ##
=================================================
- Coverage 72.8732% 56.0952% -16.7780%
=================================================
Files 1605 1737 +132
Lines 446555 626547 +179992
=================================================
+ Hits 325419 351463 +26044
- Misses 101082 251326 +150244
- Partials 20054 23758 +3704
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Signed-off-by: crazycs520 <crazycs520@gmail.com>
Signed-off-by: crazycs520 <crazycs520@gmail.com>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: crazycs520 <crazycs520@gmail.com>
Signed-off-by: crazycs520 <crazycs520@gmail.com>
Signed-off-by: crazycs520 <crazycs520@gmail.com>
Signed-off-by: crazycs520 <crazycs520@gmail.com>
/retest-required |
@crazycs520: Cannot trigger testing until a trusted user reviews the PR and leaves an In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
…armup get tso Signed-off-by: crazycs520 <crazycs520@gmail.com>
Signed-off-by: crazycs520 <crazycs520@gmail.com>
What problem does this PR solve?
Issue Number: close #56018
Problem Summary: try use max uint64 as tso for table/index reader which only has 1 cop task and in autocommit txn.
What changed and how does it work?
Skip wait tso when table/index reader only has 1 cop task and in auto-commit transaction.
Drawback
This change may break linearizability, so this optimization can only be used when the
tidb_guarantee_linearizability
is disabled.Sysbench
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.