keith-turner opened a new pull request, #5530:
URL: https://github.com/apache/accumulo/pull/5530
In the case where setting up a balancer fails the manager currently falls
back to using the SimpleLoadBalancer. This fallback behavior could be very
unhelpful and turn one problem (bad balancer config) into multiple problems as
the simple load balancer may start assigning tablets in a way that is
detrimental.
The reason this is a draft is to get feedback on the approach. New test are
needed, but those will be a lot more work than this simple change. Did not want
to write test if we settle on a different approach. Need to test the following
for system balancer and per table balancers.
* Configuring a balancer class name that does not exists
* Configuring a balancer class that is not of the correct type
* Configuring a balancer class that throws an exception in it constructor
* Configuring a balancer class that throws an exception when init is called
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
