-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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
tidb: always rebuild plan for retry. #5219
Conversation
Fixs record and index inconsistency bug.
/run-all-tests tidb-test=release-1.0 |
@@ -168,6 +168,7 @@ type ExecuteStmt struct { | |||
|
|||
Name string | |||
UsingVars []ExprNode | |||
ExecID uint32 |
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.
When will this ExecID be used?
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.
To build Execute
plan.
ctx context.Context | ||
StmtNode ast.StmtNode | ||
|
||
Ctx context.Context |
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.
Why export it?
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.
It will be constructed by another package.
LGTM |
1 similar comment
LGTM |
Fixs record and index inconsistency bug.