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)) {
