Skip to content

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

Merged
morningman merged 1 commit into
apache:masterfrom
stdpain:fix_colocate_balance_hang
Dec 16, 2020
Merged

[BUG] Fix colocate balance bug when no available BE#5079
morningman merged 1 commit into
apache:masterfrom
stdpain:fix_colocate_balance_hang

Conversation

@stdpain

@stdpainstdpain commented Dec 14, 2020•
edited
Loading

Copy link
Copy Markdown
Contributor

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
stdpainforce-pushed the fix_colocate_balance_hang branch from 7331aec to c27920bCompareDecember 14, 2020 11:36
morningman
morningman reviewed Dec 14, 2020
View reviewed changes
Comment threadfe/fe-core/src/test/java/org/apache/doris/clone/ColocateTableBalancerTest.java Outdated
@morningmanmorningman added the kind/fix Categorizes issue or PR as related to a bug. label Dec 14, 2020
@EmmyMiao87EmmyMiao87 added the area/colocated Issues or PRs related to colocated tables label Dec 14, 2020
@stdpain
stdpainforce-pushed the fix_colocate_balance_hang branch from c27920b to 9e97274CompareDecember 14, 2020 12:17
morningman
morningman approved these changes Dec 15, 2020
View reviewed changes

@morningmanmorningman left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Dec 15, 2020
@morningmanmorningman mentioned this pull request Dec 15, 2020
Closed
2 tasks
@morningman
morningman merged commit ef15c51 into apache:masterDec 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
@yangzhgyangzhg 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_boundariesapache…
0b91a43
…#56935 (apache#5079)
pick apache#56935
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morningmanmorningmanmorningman approved these changes

Assignees

No one assigned

Labels

approvedIndicates a PR has been approved by one committer.area/colocatedIssues or PRs related to colocated tableskind/fixCategorizes 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