[accumulo] branch main updated: Fix trivial warnings (#2291)

[ct...@apache.org]

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

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


The following commit(s) were added to refs/heads/main by this push:
     new d4cb0eb  Fix trivial warnings (#2291)
d4cb0eb is described below

commit d4cb0ebacddcad11a520e87829714f3f67dc71d5
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Mon Sep 27 21:02:31 2021 -0400

    Fix trivial warnings (#2291)
    
    * Remove unused import
    * Remove unused log field in test class
    * Remove misleading javadoc about renamed constraint check method that
      wasn't renamed
---
 .../java/org/apache/accumulo/core/data/constraints/Constraint.java    | 4 ----
 .../src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java   | 4 ----
 .../java/org/apache/accumulo/tserver/tablet/CompactableUtils.java     | 1 -
 3 files changed, 9 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/core/data/constraints/Constraint.java b/core/src/main/java/org/apache/accumulo/core/data/constraints/Constraint.java
index 02068a4..5ce7840 100644
--- a/core/src/main/java/org/apache/accumulo/core/data/constraints/Constraint.java
+++ b/core/src/main/java/org/apache/accumulo/core/data/constraints/Constraint.java
@@ -90,10 +90,6 @@ public interface Constraint {
    *
    * Violation codes must be non-negative. Negative violation codes are reserved for system use.
    *
-   * New API equivalent of
-   * {@link org.apache.accumulo.core.constraints.Constraint#check(org.apache.accumulo.core.constraints.Constraint.Environment, Mutation)}
-   * but renamed to prevent ambiguous method call errors.
-   *
    * @param env
    *          constraint environment
    * @param mutation
diff --git a/server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java b/server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java
index c6736ed..ecb216f 100644
--- a/server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java
+++ b/server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java
@@ -39,13 +39,9 @@ import org.apache.accumulo.core.data.TableId;
 import org.apache.accumulo.server.replication.StatusUtil;
 import org.apache.accumulo.server.replication.proto.Replication.Status;
 import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class GarbageCollectionTest {
 
-  private static final Logger log = LoggerFactory.getLogger(GarbageCollectionTest.class);
-
   static class TestGCE implements GarbageCollectionEnvironment {
     TreeSet<String> candidates = new TreeSet<>();
     ArrayList<String> blips = new ArrayList<>();
diff --git a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableUtils.java b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableUtils.java
index c625263..742180f 100644
--- a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableUtils.java
+++ b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/CompactableUtils.java
@@ -72,7 +72,6 @@ import org.apache.accumulo.core.summary.SummaryCollection;
 import org.apache.accumulo.core.summary.SummaryReader;
 import org.apache.accumulo.core.util.Pair;
 import org.apache.accumulo.server.ServiceEnvironmentImpl;
-import org.apache.accumulo.server.compaction.CompactionInfo;
 import org.apache.accumulo.server.compaction.CompactionStats;
 import org.apache.accumulo.server.compaction.FileCompactor;
 import org.apache.accumulo.server.compaction.FileCompactor.CompactionCanceledException;