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()));
     }