[accumulo] branch 1451-external-compactions-feature updated: Removed remaining CBUG and unnecessary TODO

[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 7e43857  Removed remaining CBUG and unnecessary TODO
7e43857 is described below

commit 7e438575baaa3295352567998fdcbec80902b30b
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Tue May 11 12:56:01 2021 +0000

    Removed remaining CBUG and unnecessary TODO
---
 .../main/java/org/apache/accumulo/compactor/Compactor.java  |  1 -
 .../org/apache/accumulo/test/performance/NullTserver.java   | 13 ++-----------
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git 
a/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java 
b/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
index 9461c19..bfecade 100644
--- 
a/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
+++ 
b/server/compactor/src/main/java/org/apache/accumulo/compactor/Compactor.java
@@ -584,7 +584,6 @@ public class Compactor extends AbstractServer
           throw new RuntimeException("Compaction failed", e);
         } finally {
           stopped.countDown();
-          // CBUG Any cleanup
         }
       }
     };
diff --git 
a/test/src/main/java/org/apache/accumulo/test/performance/NullTserver.java 
b/test/src/main/java/org/apache/accumulo/test/performance/NullTserver.java
index 9abb4b0..7f2cc14 100644
--- a/test/src/main/java/org/apache/accumulo/test/performance/NullTserver.java
+++ b/test/src/main/java/org/apache/accumulo/test/performance/NullTserver.java
@@ -284,7 +284,6 @@ public class NullTserver {
     @Override
     public List<TCompactionQueueSummary> getCompactionQueueInfo(TInfo tinfo,
         TCredentials credentials) throws ThriftSecurityException, TException {
-      // TODO Auto-generated method stub
       return null;
     }
 
@@ -292,29 +291,21 @@ public class NullTserver {
     public TExternalCompactionJob reserveCompactionJob(TInfo tinfo, 
TCredentials credentials,
         String queueName, long priority, String compactor, String 
externalCompactionId)
         throws ThriftSecurityException, TException {
-      // TODO Auto-generated method stub
       return null;
     }
 
     @Override
     public void compactionJobFinished(TInfo tinfo, TCredentials credentials,
         String externalCompactionId, TKeyExtent extent, long fileSize, long 
entries)
-        throws ThriftSecurityException, TException {
-      // TODO Auto-generated method stub
-
-    }
+        throws ThriftSecurityException, TException {}
 
     @Override
     public void compactionJobFailed(TInfo tinfo, TCredentials credentials,
-        String externalCompactionId, TKeyExtent extent) throws TException {
-      // TODO Auto-generated method stub
-
-    }
+        String externalCompactionId, TKeyExtent extent) throws TException {}
 
     @Override
     public boolean isRunningExternalCompaction(TInfo tinfo, TCredentials 
credentials,
         String externalCompactionId, TKeyExtent extent) throws TException {
-      // TODO Auto-generated method stub
       return false;
     }
   }