[accumulo] branch master updated: Fix trivial warnings

[ct...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new d920142  Fix trivial warnings
d920142 is described below

commit d92014212f318475ee1e1877e20b5a35c84e2a13
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Tue May 14 17:10:51 2019 -0400

    Fix trivial warnings
---
 core/src/main/java/org/apache/accumulo/fate/util/Retry.java          | 5 -----
 .../apache/accumulo/test/functional/ConfigurableCompactionIT.java    | 2 +-
 test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java  | 4 ++--
 3 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/core/src/main/java/org/apache/accumulo/fate/util/Retry.java b/core/src/main/java/org/apache/accumulo/fate/util/Retry.java
index cc679e9..4679400 100644
--- a/core/src/main/java/org/apache/accumulo/fate/util/Retry.java
+++ b/core/src/main/java/org/apache/accumulo/fate/util/Retry.java
@@ -291,16 +291,11 @@ public class Retry {
 
   public interface NeedsBackOffFactor {
     /**
-     *
      * @param backOffFactor
      *          the number that the wait increment will be successively multiplied by to make the
      *          time between retries to be exponentially increasing. The default value will be one.
-     *
-     * @return
      */
-
     NeedsLogInterval backOffFactor(double backOffFactor);
-
   }
 
   public interface NeedsLogInterval {
diff --git a/test/src/main/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java b/test/src/main/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java
index 4cfb5ce..ddf135a 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/ConfigurableCompactionIT.java
@@ -114,7 +114,7 @@ public class ConfigurableCompactionIT extends ConfigurableMacBase {
           installJar(getCluster().getConfig().getAccumuloDir(), "/TestCompactionStrat.jar");
       c.instanceOperations().setProperty(
           Property.VFS_CONTEXT_CLASSPATH_PROPERTY.getKey() + "context1", destFile.toString());
-      Map props = new HashMap<>();
+      Map<String,String> props = new HashMap<>();
       props.put(Property.TABLE_MAJC_RATIO.getKey(), "10");
       props.put(Property.TABLE_CLASSPATH.getKey(), "context1");
       // EfgCompactionStrat will only compact a tablet w/ end row of 'efg'. No other tablets are
diff --git a/test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java b/test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java
index d987d9d..c8c53d6 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/SplitIT.java
@@ -126,7 +126,7 @@ public class SplitIT extends AccumuloClusterHarness {
     try (AccumuloClient c = Accumulo.newClient().from(getClientProps()).build()) {
       String table = getUniqueNames(1)[0];
 
-      Map props = new HashMap<>();
+      Map<String,String> props = new HashMap<>();
       props.put(Property.TABLE_SPLIT_THRESHOLD.getKey(), "256K");
       props.put(Property.TABLE_FILE_COMPRESSED_BLOCK_SIZE.getKey(), "1K");
 
@@ -165,7 +165,7 @@ public class SplitIT extends AccumuloClusterHarness {
     try (AccumuloClient c = Accumulo.newClient().from(getClientProps()).build()) {
       String tableName = getUniqueNames(1)[0];
 
-      Map props = new HashMap<>();
+      Map<String,String> props = new HashMap<>();
       props.put(Property.TABLE_SPLIT_THRESHOLD.getKey(), "10K");
       props.put(Property.TABLE_FILE_COMPRESSION_TYPE.getKey(), "none");