[accumulo] branch 1451-external-compactions-feature updated: Fix some QA issues

[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 8ab81b0  Fix some QA issues
8ab81b0 is described below

commit 8ab81b0d2cfab779e0239a951d3286aed9493658
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Wed May 5 14:26:06 2021 +0000

    Fix some QA issues
---
 .../org/apache/accumulo/core/client/admin/InstanceOperations.java     | 2 --
 .../src/main/java/org/apache/accumulo/compactor/Compactor.java        | 4 ++++
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git 
a/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java
 
b/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java
index 6bb40ed..6e9df1e 100644
--- 
a/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java
+++ 
b/core/src/main/java/org/apache/accumulo/core/client/admin/InstanceOperations.java
@@ -119,8 +119,6 @@ public interface InstanceOperations {
    * List all compactions running in Accumulo
    *
    * @return the list of active compactions
-   * @throws AccumuloSecurityException
-   * @throws AccumuloException
    * @since 2.1.0
    */
   List<ActiveCompaction> getActiveCompactions() throws AccumuloException, 
AccumuloSecurityException;
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 23bff9f..c19e1d4 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
@@ -92,6 +92,8 @@ import org.slf4j.LoggerFactory;
 
 import com.beust.jcommander.Parameter;
 
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
+
 public class Compactor extends AbstractServer
     implements org.apache.accumulo.core.compaction.thrift.Compactor.Iface {
 
@@ -559,6 +561,8 @@ public class Compactor extends AbstractServer
     return supplier;
   }
 
+  @SuppressFBWarnings(value = "PREDICTABLE_RANDOM",
+      justification = "random used for jitter, not security functions")
   protected long getWaitTimeBetweenCompactionChecks() {
     // get the total number of compactors assigned to this queue
     int numCompactors = ExternalCompactionUtil.countCompactors(queueName, 
getContext());