Skip to content

[BUG] Fix colocate balance bug when no available BE #5079

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
Dec 16, 2020
Merged

[BUG] Fix colocate balance bug when no available BE #5079

morningman merged 1 commit into from
Dec 16, 2020

Conversation

@stdpain
Copy link
Contributor

@stdpain stdpain commented Dec 14, 2020 •
edited
Loading

Proposed changes

will close #5075

Types of changes

What types of changes does your code introduce to Doris?

  • Bugfix (Fix colocate balance bug when no available BE)
  • Code refactor (Improve code readability)

Checklist

  • I have create an issue on (Fix [BUG] FE CPU 100% #5075 ), and have described the bug/feature there in detail
  • Compiling and unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged

Further comments

When no avaliable BE，colocate rebalance will hang

@stdpain stdpain force-pushed the fix_colocate_balance_hang branch from 7331aec to c27920b Compare December 14, 2020 11:36
@morningman morningman added the kind/fix Categorizes issue or PR as related to a bug. label Dec 14, 2020
@EmmyMiao87 EmmyMiao87 added the area/colocated Issues or PRs related to colocated tables label Dec 14, 2020
@stdpain stdpain force-pushed the fix_colocate_balance_hang branch from c27920b to 9e97274 Compare December 14, 2020 12:17
@stdpain stdpain requested a review from morningman December 14, 2020 12:19
morningman
morningman approved these changes Dec 15, 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 the approved Indicates a PR has been approved by one committer. label Dec 15, 2020
@morningman morningman mentioned this pull request Dec 15, 2020
Closed
2 tasks
@morningman morningman merged commit ef15c51 into apache:master Dec 16, 2020
@stdpain stdpain deleted the fix_colocate_balance_hang branch December 18, 2020 08:30
morningman pushed a commit to baidu-doris/incubator-doris that referenced this pull request Jan 5, 2021
@stdpain @morningman-cmy
[BUG] Fix colocate balance bug when no available BE (apache#5079)
82f74e5
@yangzhg yangzhg mentioned this pull request Feb 9, 2021
Closed
Hastyshell pushed a commit to Hastyshell/doris that referenced this pull request Nov 12, 2025
@gavinchou
[fix](cloud) Fix unhandled error when get_partition_boundaries apache…
0b91a43 
…#56935 (apache#5079)

pick apache#56935
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.

[BUG] FE CPU 100%

3 participants

@stdpain @morningman @EmmyMiao87