(accumulo) branch elasticity updated: removes todo that was completed in #4394

[kturner]

This is an automated email from the ASF dual-hosted git repository.

kturner pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/elasticity by this push:
     new c9bb7e3963 removes todo that was completed in #4394
c9bb7e3963 is described below

commit c9bb7e396370ee2710bee37179774c3f7fd93584
Author: Keith Turner <ktur...@apache.org>
AuthorDate: Thu May 30 17:30:20 2024 -0400

    removes todo that was completed in #4394
---
 .../main/java/org/apache/accumulo/manager/TabletGroupWatcher.java   | 6 ------
 1 file changed, 6 deletions(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
index 9b65fcdc57..aa77ee6fc5 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
@@ -605,12 +605,6 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
         manager.getCompactionCoordinator().addJobs(tm, jobs);
       }
 
-      // ELASITICITY_TODO the case where a planner generates compactions at 
time T1 for tablet
-      // and later at time T2 generates nothing for the same tablet is not 
being handled. At
-      // time T1 something could have been queued. However at time T2 we will 
not clear those
-      // entries from the queue because we see nothing here for that case. 
After a full
-      // metadata scan could remove any tablets that were not updated during 
the scan.
-
       if (actions.contains(ManagementAction.NEEDS_LOCATION_UPDATE)
           || actions.contains(ManagementAction.NEEDS_RECOVERY)) {