[accumulo] branch master updated: Remove warnings from #1495

[ctubbsii]

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 2b0b971  Remove warnings from #1495
2b0b971 is described below

commit 2b0b9710bea9e06ad56a6e8fc317b06adbdcd44d
Author: Christopher Tubbs <ctubb...@apache.org>
AuthorDate: Thu Feb 20 16:10:05 2020 -0500

    Remove warnings from #1495
    
    Remove unused variable warnings
---
 .../main/java/org/apache/accumulo/test/functional/CleanUpIT.java   | 7 +++----
 .../java/org/apache/accumulo/test/functional/PermissionsIT.java    | 3 ++-
 2 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java 
b/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java
index 7890d4b..34309cd 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java
@@ -39,6 +39,8 @@ import org.junit.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import com.google.common.collect.Iterables;
+
 /**
  * Ensures that all threads spawned for ZooKeeper and Thrift connectivity are 
reaped after calling
  * CleanUp.shutdown().
@@ -130,10 +132,7 @@ public class CleanUpIT extends SharedMiniClusterBase {
       }
 
       try {
-        count = 0;
-        for (Entry<Key,Value> keyValueEntry : scanner) {
-          count++;
-        }
+        count = Iterables.size(scanner);
         fail("scanner did not fail");
       } catch (Exception e) {
 
diff --git 
a/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java 
b/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java
index 9c5f6bc..1e6418c 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java
@@ -20,6 +20,7 @@ package org.apache.accumulo.test.functional;
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
@@ -722,7 +723,7 @@ public class PermissionsIT extends AccumuloClusterHarness {
       case READ:
         try (Scanner scanner = test_user_client.createScanner(tableName, 
Authorizations.EMPTY)) {
           for (Entry<Key,Value> keyValueEntry : scanner) {
-            // empty
+            assertNotNull(keyValueEntry);
           }
         }
         break;