-
Notifications
You must be signed in to change notification settings - Fork 721
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
*: update the error dependency and do the corresponding changes #2760
Conversation
b0470b7
to
fcb2368
Compare
5a4872d
to
cd4036b
Compare
Signed-off-by: Ryan Leung <rleungx@gmail.com>
Signed-off-by: Ryan Leung <rleungx@gmail.com>
cd4036b
to
5359dfb
Compare
Signed-off-by: Ryan Leung <rleungx@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@Yisaer,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: scheduling(slack). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@JmPotato,Thanks for your review. However, LGTM is restricted to Reviewers or higher roles.See the corresponding SIG page for more information. Related SIGs: scheduling(slack). |
LGTM |
/merge |
@nolouch Oops! This PR requires at least 2 LGTMs to merge. The current number of |
/merge |
@nolouch Oops! This PR requires at least 2 LGTMs to merge. The current number of |
/merge |
@nolouch Oops! This PR requires at least 2 LGTMs to merge. The current number of |
/merge |
/run-all-tests |
@rleungx merge failed. |
/merge |
/run-all-tests |
…#2760) Signed-off-by: Ryan Leung <rleungx@gmail.com> Signed-off-by: ZenoTan <zenotan1998@gmail.com>
…#2760) Signed-off-by: Ryan Leung <rleungx@gmail.com>
…#2760) Signed-off-by: Ryan Leung <rleungx@gmail.com>
/run-cherry-picker |
Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
cherry pick to release-4.0 in PR #2902 |
What problem does this PR solve?
Related to #2704.
What is changed and how it works?
This PR updates the way to define an error and uses
Wrap
andFastGenWithCause
to output the internal error.Check List
Tests
Release note