[accumulo] branch 1451-external-compactions-feature updated: Removed TODO in CompactableImpl

[dlmarion]

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

dlmarion pushed a commit to branch 1451-external-compactions-feature
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to 
refs/heads/1451-external-compactions-feature by this push:
     new 83b47db  Removed TODO in CompactableImpl
83b47db is described below

commit 83b47db775a9e4d4f0105d1b3ff0c77a499a6ca1
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Mon May 10 17:16:34 2021 +0000

    Removed TODO in CompactableImpl
---
 .../org/apache/accumulo/tserver/tablet/CompactableImpl.java   | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
index 58dc4dc..695ccae 100644
--- 
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
+++ 
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableImpl.java
@@ -94,6 +94,11 @@ import com.google.common.collect.Sets;
  */
 public class CompactableImpl implements Compactable {
 
+  private static class ExternalCompactionInfo {
+    ExternalCompactionMetadata meta;
+    CompactionJob job;
+  }
+
   private static final Logger log = 
LoggerFactory.getLogger(CompactableImpl.class);
 
   private final Tablet tablet;
@@ -132,12 +137,6 @@ public class CompactableImpl implements Compactable {
 
   private volatile boolean closed = false;
 
-  // TODO move to top of class
-  private static class ExternalCompactionInfo {
-    ExternalCompactionMetadata meta;
-    CompactionJob job;
-  }
-
   private Map<ExternalCompactionId,ExternalCompactionInfo> externalCompactions 
=
       new ConcurrentHashMap<>();