You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by jm...@apache.org on 2022/10/12 17:36:32 UTC
[accumulo] branch main updated: Remove unneeded TODO in TabletBalancer (#3018)
This is an automated email from the ASF dual-hosted git repository.
jmark99 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/main by this push:
new 74155ba491 Remove unneeded TODO in TabletBalancer (#3018)
74155ba491 is described below
commit 74155ba4911e7146dbcb64eb5888441b447ed2c0
Author: Mark Owens <jm...@apache.org>
AuthorDate: Wed Oct 12 13:36:27 2022 -0400
Remove unneeded TODO in TabletBalancer (#3018)
Trivial change to remove TODO comments within the TabletBalancer.java 'run' method.
The TODO references an older Jira ticket ACCUMUO-2938 that has been marked resolved and closed. The TODO is no longer relevant.
---
.../org/apache/accumulo/server/master/balancer/TabletBalancer.java | 4 ----
1 file changed, 4 deletions(-)
diff --git a/server/base/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java b/server/base/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
index 0db9d6521c..23a5f5d679 100644
--- a/server/base/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
+++ b/server/base/src/main/java/org/apache/accumulo/server/master/balancer/TabletBalancer.java
@@ -213,10 +213,6 @@ public abstract class TabletBalancer
@Override
public void run() {
balancerLog.warn("Not balancing due to {} outstanding migrations.", migrations.size());
- /*
- * TODO ACCUMULO-2938 redact key extents in this output to avoid leaking protected
- * information.
- */
balancerLog.debug("Sample up to 10 outstanding migrations: {}",
migrations.stream().limit(10).collect(toList()));
}