Skip to content

[Colocation] Fix Colocation balance endless loop bug #4471

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.

Sign up for GitHub

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

Jump to bottom
Merged
morningman merged 1 commit into from
Aug 28, 2020
Merged

[Colocation] Fix Colocation balance endless loop bug #4471

morningman merged 1 commit into from
Aug 28, 2020

Conversation

@wyb
Copy link
Contributor

@wyb wyb commented Aug 27, 2020

  1. Only one available backend.
  2. All backends are checked but this round is not changed. For example, all backends are on the same host.

morningman
morningman approved these changes Aug 27, 2020
View reviewed changes
Copy link
Contributor

@morningman morningman left a comment

Choose a reason for hiding this comment

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

LGTM

@morningman morningman added approved Indicates a PR has been approved by one committer. area/colocated Issues or PRs related to colocated tables kind/fix Categorizes issue or PR as related to a bug. labels Aug 27, 2020
@morningman morningman merged commit ec64789 into apache:master Aug 28, 2020
@yangzhg yangzhg mentioned this pull request Feb 9, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningman morningman morningman approved these changes

Assignees

No one assigned

Labels

approved Indicates a PR has been approved by one committer. area/colocated Issues or PRs related to colocated tables kind/fix Categorizes issue or PR as related to a bug.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wyb @morningman