You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2014/08/11 23:41:10 UTC
[07/12] git commit: ACCUMULO-3058 Remove broken javadoc line
ACCUMULO-3058 Remove broken javadoc line
Removes incorrect instructions referencing non-existent methods.
Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/814414a8
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/814414a8
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/814414a8
Branch: refs/heads/1.6.1-SNAPSHOT
Commit: 814414a8296a6186cc4296126440dde8c797862f
Parents: ec53191
Author: Christopher Tubbs <ct...@apache.org>
Authored: Mon Aug 11 16:51:50 2014 -0400
Committer: Christopher Tubbs <ct...@apache.org>
Committed: Mon Aug 11 16:51:50 2014 -0400
----------------------------------------------------------------------
.../server/master/balancer/TabletBalancer.java | 15 +++++++--------
1 file changed, 7 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/accumulo/blob/814414a8/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
----------------------------------------------------------------------
diff --git a/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java b/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
index 16c5dbc..87f8333 100644
--- a/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
+++ b/server/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
@@ -23,9 +23,6 @@ import java.util.Map;
import java.util.Set;
import java.util.SortedMap;
-import com.google.common.collect.Iterables;
-
-import org.apache.accumulo.trace.instrument.Tracer;
import org.apache.accumulo.core.client.impl.thrift.ThriftSecurityException;
import org.apache.accumulo.core.data.KeyExtent;
import org.apache.accumulo.core.master.thrift.TabletServerStatus;
@@ -37,10 +34,13 @@ import org.apache.accumulo.server.conf.ServerConfiguration;
import org.apache.accumulo.server.master.state.TServerInstance;
import org.apache.accumulo.server.master.state.TabletMigration;
import org.apache.accumulo.server.security.SecurityConstants;
+import org.apache.accumulo.trace.instrument.Tracer;
import org.apache.log4j.Logger;
import org.apache.thrift.TException;
import org.apache.thrift.transport.TTransportException;
+import com.google.common.collect.Iterables;
+
public abstract class TabletBalancer {
private static final Logger log = Logger.getLogger(TabletBalancer.class);
@@ -70,11 +70,10 @@ public abstract class TabletBalancer {
/**
* Ask the balancer if any migrations are necessary.
- *
- * If the balancer is going to self-abort due to some environmental constraint (e.g. it requires some minimum number of tservers, or a maximum number
- * of outstanding migrations), it should issue a log message to alert operators. The message should be at WARN normally and at ERROR if the balancer knows that the
- * problem can not self correct. It should not issue these messages more than once a minute. Subclasses can use the convenience methods of {@link #constraintNotMet()} and
- * {@link #balanceSuccessful()} to accomplish this logging.
+ *
+ * If the balancer is going to self-abort due to some environmental constraint (e.g. it requires some minimum number of tservers, or a maximum number of
+ * outstanding migrations), it should issue a log message to alert operators. The message should be at WARN normally and at ERROR if the balancer knows that
+ * the problem can not self correct. It should not issue these messages more than once a minute.
*
* @param current
* The current table-summary state of all the online tablet servers. Read-only.