-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
executor: fix issue of txn_ts of stale-read query is 0 in slow log #52484
Conversation
Signed-off-by: crazycs520 <crazycs520@gmail.com>
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/test-infra repository. |
/check-issue-triage-complete |
Can we consider distinguishing the start_ts allocated from PD and the stale read ts that's calculated in other way, to avoid possible misunderstanding (e.g. the ts might be the same with another transaction's start_ts, and one may think that the statement is part of that transaction)? |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: zyguan The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
[LGTM Timeline notifier]Timeline:
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #52484 +/- ##
================================================
+ Coverage 72.2050% 74.2928% +2.0877%
================================================
Files 1467 1467
Lines 427020 427484 +464
================================================
+ Hits 308330 317590 +9260
+ Misses 99515 90046 -9469
- Partials 19175 19848 +673
Flags with carried forward coverage won't be shown. Click here to find out more.
|
#52494 has a better fix than this. |
What problem does this PR solve?
Issue Number: close #45545
Problem Summary: fix issue of txn_ts of stale-read query is 0 in slow log
What changed and how does it work?
Before This PR
The slow log of stale-read query is the following,
Txn_start_ts
is 0, which is unexpected.This PR
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.