Skip to content
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

*: add engine filter to exclude special stores (#2426) #2447

Merged
merged 5 commits into from
May 25, 2020

Conversation

sre-bot
Copy link
Contributor

@sre-bot sre-bot commented May 22, 2020

cherry-pick #2426 to release-4.0


Signed-off-by: nolouch nolouch@gmail.com

What problem does this PR solve?

Fix #2423
AvaiableStore() always returns flash nodes, so the tikv nodes cannot be used.

Release note

  • Fix the issue that presplit not work within tiflash cluster.

Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
Signed-off-by: nolouch <nolouch@gmail.com>
@sre-bot sre-bot added type/bug The issue is confirmed as a bug. type/cherry-pick labels May 22, 2020
@sre-bot sre-bot added this to the v4.0.0-rc.2 milestone May 22, 2020
@nolouch nolouch modified the milestones: v4.0.0-rc.2, v4.0.0-ga May 22, 2020
@lhy1024
Copy link
Contributor

lhy1024 commented May 25, 2020

/approve

@sre-bot
Copy link
Contributor Author

sre-bot commented May 25, 2020

Approve failed.

@zhouqiang-cl
Copy link
Contributor

/run-all-tests

Copy link
Contributor

@zhouqiang-cl zhouqiang-cl left a comment

Choose a reason for hiding this comment

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

LGTM

@jebter
Copy link
Collaborator

jebter commented May 25, 2020

/merge

@sre-bot sre-bot added the status/can-merge Indicates a PR has been approved by a committer. label May 25, 2020
@sre-bot
Copy link
Contributor Author

sre-bot commented May 25, 2020

/run-all-tests

@jebter jebter merged commit 0fb3adf into tikv:release-4.0 May 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/can-merge Indicates a PR has been approved by a committer. type/bug The issue is confirmed as a bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants