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;