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

Reply via email to