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
plan: fix a bug in rebuildRange
when enabling prepare-plan-cache.
#6637
Conversation
@@ -0,0 +1,51 @@ | |||
package plan_test |
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.
Need a license.
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.
Will this test case cover this bug? If we revert the modification in common_plans.go, will this case fail?
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 fail.
plan/errors.go
Outdated
@@ -1,100 +0,0 @@ | |||
// Copyright 2018 PingCAP, Inc. |
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 remove this file?
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's a mistake...
/run-all-tests |
plan/prepare_test.go
Outdated
@@ -0,0 +1,65 @@ | |||
// Copyright 2017 PingCAP, Inc. |
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.
s/ 2017/ 2018
I've tested this pr by TPC-C. LGTM |
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
@winoros Do we need to cherry pick this one to release-2.0? |
…ingcap#6637) * plan: fix a bug in `rebuildRange` when enabling prepare-plan-cache.
It cannot build range successfully if the plan is an index plan.
PTAL @lamxTyler @zz-jason @XuHuaiyu @jackysp