ctubbsii commented on a change in pull request #401: Fix ChaoticBalancerIT
URL: https://github.com/apache/accumulo/pull/401#discussion_r175156884
##########
File path:
test/src/test/java/org/apache/accumulo/test/functional/ChaoticBalancerIT.java
##########
@@ -42,6 +42,7 @@ public void configureMiniCluster(MiniAccumuloConfigImpl cfg,
Configuration hadoo
Map<String,String> siteConfig = cfg.getSiteConfig();
siteConfig.put(Property.TSERV_MAXMEM.getKey(), "10K");
siteConfig.put(Property.TSERV_MAJC_DELAY.getKey(), "0");
+ siteConfig.put(Property.TABLE_LOAD_BALANCER.getKey(),
ChaoticLoadBalancer.class.getName());
Review comment:
After looking at the code more, I think there might be more fundamental
problems with the ChaoticLoadBalancer, as it does not appear to be "per-table"
aware, and will compete with other balancers trying to balance their tables. It
probably is best to set this at site, like you did.... at least for now.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
