Skip to content

planner: correct estimated row count for inner plan of index join (#10015)#10044

Merged
zz-jason merged 2 commits intopingcap:release-2.1from
eurekaka:pick_index_join_inner
Apr 4, 2019
Merged

planner: correct estimated row count for inner plan of index join (#10015)#10044
zz-jason merged 2 commits intopingcap:release-2.1from
eurekaka:pick_index_join_inner

Conversation

@eurekaka
Copy link
Copy Markdown
Contributor

@eurekaka eurekaka commented Apr 4, 2019

cherry-pick #10015

@eurekaka eurekaka added type/bugfix This PR fixes a bug. sig/planner SIG: Planner type/2.1 cherry-pick labels Apr 4, 2019
@eurekaka
Copy link
Copy Markdown
Contributor Author

eurekaka commented Apr 4, 2019

/run-all-tests tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1

@eurekaka
Copy link
Copy Markdown
Contributor Author

eurekaka commented Apr 4, 2019

/run-integration-ddl-test tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1
/run-mybatis-test tidb-test=release-2.1 tikv=release-2.1 pd=release-2.1

Copy link
Copy Markdown
Contributor

@lzmhhh123 lzmhhh123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@lzmhhh123 lzmhhh123 added the status/LGT1 Indicates that a PR has LGTM 1. label Apr 4, 2019
Copy link
Copy Markdown
Member

@zz-jason zz-jason left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zz-jason zz-jason added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Apr 4, 2019
@zz-jason zz-jason merged commit d8f04bd into pingcap:release-2.1 Apr 4, 2019
@eurekaka eurekaka deleted the pick_index_join_inner branch April 4, 2019 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/planner SIG: Planner status/LGT2 Indicates that a PR has LGTM 2. type/bugfix This PR fixes a bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants