Skip to content

planner: fix uncorrect behavior of index join (#11724)#11759

Merged
eurekaka merged 1 commit into
pingcap:release-3.0from
winoros:pick
Aug 16, 2019
Merged

planner: fix uncorrect behavior of index join (#11724)#11759
eurekaka merged 1 commit into
pingcap:release-3.0from
winoros:pick

Conversation

@winoros
Copy link
Copy Markdown
Member

@winoros winoros commented Aug 16, 2019

What problem does this PR solve?

pick #11724

Check List

Tests

  • Unit test

Related changes

  • Need to be included in the release note

@winoros winoros added type/bugfix This PR fixes a bug. sig/planner SIG: Planner labels Aug 16, 2019
@winoros
Copy link
Copy Markdown
Member Author

winoros commented Aug 16, 2019

/run-all-tests

Copy link
Copy Markdown
Contributor

@eurekaka eurekaka left a comment

Choose a reason for hiding this comment

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

LGTM

@eurekaka eurekaka added the status/LGT1 Indicates that a PR has LGTM 1. label Aug 16, 2019
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 status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Aug 16, 2019
@sre-bot
Copy link
Copy Markdown
Contributor

sre-bot commented Aug 16, 2019

Sorry @lzmhhh123, you don't have permission to trigger auto merge event on this branch.

@eurekaka eurekaka merged commit 43aaf9a into pingcap:release-3.0 Aug 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sig/planner SIG: Planner status/can-merge Indicates a PR has been approved by a committer. 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.

4 participants