-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20525/
-----------------------------------------------------------
(Updated April 21, 2014, 9:32 p.m.)
Review request for accumulo, Eric Newton and Mike Drob.
Changes
-------
Limit the frequency of warnings to avoid transient blockage and flooding
messages.
Previous patch:
$ grep -i "WARN : not balancing" logs/host1-9873/master_host1.example.com.log |
grep -i "outstanding migrations" | wc -l
816
And now:
$ grep -i "WARN : not balancing" logs/host1-19121/master_host1.example.com.log
| grep -i "outstanding migrations" | wc -l
12
The test is set to use per-table balancers and has 3 tables, so that 12 count
represents four minutes with a set of 3 messages per minute.
Bugs: ACCUMULO-2694
https://issues.apache.org/jira/browse/ACCUMULO-2694
Repository: accumulo
Description
-------
ACCUMULO-2694 Fix handling of tablet migrations for offline tables.
* Adds a funtional test that fails due to not rebalancing
* Fix master to clear migrations when it learns that a table has gone
offline
* Update master to periodically clean up migrations for offline tables
* Fix balancers to make sure they log if they can't balance.
Diffs (updated)
-----
src/server/pom.xml dbe4fb4
src/server/src/main/java/org/apache/accumulo/server/master/Master.java
fb7be51
src/server/src/main/java/org/apache/accumulo/server/master/balancer/ChaoticLoadBalancer.java
02a4e89
src/server/src/main/java/org/apache/accumulo/server/master/balancer/DefaultLoadBalancer.java
4826097
src/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
ad62360
test/system/auto/stress/migrations.py d07d7a8
Diff: https://reviews.apache.org/r/20525/diff/
Testing
-------
Ran functional test without other changes -> failed. After full patch
functional test passes.
Thanks,
Sean Busbey
