-
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
driver: convert ErrQueryInterrupted to actual error types #50517
driver: convert ErrQueryInterrupted to actual error types #50517
Conversation
Hi @ekexium. 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. |
fad4165
to
052e0f4
Compare
Signed-off-by: ekexium <eke@fastmail.com>
052e0f4
to
50aae2e
Compare
/retest |
@ekexium: 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/test-infra repository. |
Signed-off-by: ekexium <eke@fastmail.com>
…or-with-signal Signed-off-by: ekexium <eke@fastmail.com>
Signed-off-by: ekexium <eke@fastmail.com>
0047c36
to
9bd7f36
Compare
/retest |
@ekexium: 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/test-infra repository. |
Signed-off-by: ekexium <eke@fastmail.com>
9bd7f36
to
871f142
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #50517 +/- ##
================================================
+ Coverage 70.5606% 72.2136% +1.6530%
================================================
Files 1466 1468 +2
Lines 434758 447112 +12354
================================================
+ Hits 306768 322876 +16108
+ Misses 108716 103822 -4894
- Partials 19274 20414 +1140
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/test unit-test |
@ekexium: 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/test-infra repository. |
Signed-off-by: ekexium <eke@fastmail.com>
93b3150
to
792122b
Compare
/retest |
@ekexium: 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/test-infra repository. |
/ok-to-test |
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
In response to a cherrypick label: new pull request created to branch |
Signed-off-by: ti-chi-bot <ti-community-prow-bot@tidb.io>
In response to a cherrypick label: new pull request created to branch |
What problem does this PR solve?
Issue Number: ref #49643
Problem Summary:
When a kill signal is sent to client, and client interrupts. The error returned is a ErrQueryInterrupted. TiDB needs to convert it into concrete types.
What changed and how does it work?
Check List
Tests
Side effects
Documentation
Release note
Please refer to Release Notes Language Style Guide to write a quality release note.