Skip to content

[Backport] Coordinator fix balance to try to move max segments instead of up to max segments#5974

Closed
jihoonson wants to merge 1 commit intoapache:0.12.2from
jihoonson:backport-5927-to-0.12.2
Closed

[Backport] Coordinator fix balance to try to move max segments instead of up to max segments#5974
jihoonson wants to merge 1 commit intoapache:0.12.2from
jihoonson:backport-5927-to-0.12.2

Conversation

@jihoonson
Copy link
Copy Markdown
Contributor

Backport of #5927 to 0.12.2.

…max segments (apache#5927)

* fix move to try to move max segments instead of "up to" max segments

* fix

* fix oops
@jihoonson jihoonson added this to the 0.12.2 milestone Jul 7, 2018
@drcrallen
Copy link
Copy Markdown
Contributor

oh noes, I don't have privileges to restart specific travis tests anymore

@jihoonson
Copy link
Copy Markdown
Contributor Author

@drcrallen thanks for checking. I restarted.

It looks that only some committers have been added to the Apache Druid committers of Github. Did you set your Gitbox account?

@gianm
Copy link
Copy Markdown
Contributor

gianm commented Jul 9, 2018

@drcrallen you should be able to if you go through GitBox setup: https://gitbox.apache.org/setup/

@jihoonson
Copy link
Copy Markdown
Contributor Author

I think it would be better to not backport this because it looks not having a huge improvement on segment balancing to be worth to wait for #5981 to be fixed.

@gianm
Copy link
Copy Markdown
Contributor

gianm commented Jul 9, 2018

@jihoonson, that sounds good, let's move #5927 outside of 0.12.2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants