accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [2/2] accumulo git commit: ACCUMULO-3948 Cleanup unused variables
Date Thu, 10 Dec 2015 15:42:07 GMT
ACCUMULO-3948 Cleanup unused variables

* Remove unused variables in ShellServerIT and ScannerContextIT.
* Retain side-effects.


Project: http://git-wip-us.apache.org/repos/asf/accumulo/repo
Commit: http://git-wip-us.apache.org/repos/asf/accumulo/commit/24edc84a
Tree: http://git-wip-us.apache.org/repos/asf/accumulo/tree/24edc84a
Diff: http://git-wip-us.apache.org/repos/asf/accumulo/diff/24edc84a

Branch: refs/heads/master
Commit: 24edc84a90d39e6bcdd4a22f63d0c926084779b9
Parents: 543e314
Author: Christopher Tubbs <ctubbsii@apache.org>
Authored: Thu Dec 10 10:17:17 2015 -0500
Committer: Christopher Tubbs <ctubbsii@apache.org>
Committed: Thu Dec 10 10:22:25 2015 -0500

----------------------------------------------------------------------
 .../main/java/org/apache/accumulo/test/ShellServerIT.java    | 8 ++++----
 .../apache/accumulo/test/functional/ScannerContextIT.java    | 2 +-
 2 files changed, 5 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/24edc84a/test/src/main/java/org/apache/accumulo/test/ShellServerIT.java
----------------------------------------------------------------------
diff --git a/test/src/main/java/org/apache/accumulo/test/ShellServerIT.java b/test/src/main/java/org/apache/accumulo/test/ShellServerIT.java
index 1b100f2..39aa340 100644
--- a/test/src/main/java/org/apache/accumulo/test/ShellServerIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/ShellServerIT.java
@@ -1737,9 +1737,9 @@ public class ShellServerIT extends SharedMiniClusterBase {
     setupFakeContextPath();
     // Add the context to the table so that setscaniter works. After setscaniter succeeds,
then
     // remove the property from the table.
-    String output = ts.exec("config -s " + Property.VFS_CONTEXT_CLASSPATH_PROPERTY + FAKE_CONTEXT
+ "=" + FAKE_CONTEXT_CLASSPATH);
-    output = ts.exec("config -t t -s table.classpath.context=" + FAKE_CONTEXT);
-    output = ts.exec("setscaniter -n reverse -t t -p 21 -class org.apache.accumulo.test.functional.ValueReversingIterator");
+    ts.exec("config -s " + Property.VFS_CONTEXT_CLASSPATH_PROPERTY + FAKE_CONTEXT + "=" +
FAKE_CONTEXT_CLASSPATH);
+    ts.exec("config -t t -s table.classpath.context=" + FAKE_CONTEXT);
+    ts.exec("setscaniter -n reverse -t t -p 21 -class org.apache.accumulo.test.functional.ValueReversingIterator");
     String result = ts.exec("scan -np -b row1 -e row1");
     assertEquals(2, result.split("\n").length);
     log.error(result);
@@ -1760,7 +1760,7 @@ public class ShellServerIT extends SharedMiniClusterBase {
     assertTrue(result.contains("value"));
 
     setupRealContextPath();
-    output = ts.exec("config -s " + Property.VFS_CONTEXT_CLASSPATH_PROPERTY + REAL_CONTEXT
+ "=" + REAL_CONTEXT_CLASSPATH);
+    ts.exec("config -s " + Property.VFS_CONTEXT_CLASSPATH_PROPERTY + REAL_CONTEXT + "=" +
REAL_CONTEXT_CLASSPATH);
     result = ts.exec("scan -np -b row1 -e row1 -cc " + REAL_CONTEXT);
     log.error(result);
     assertEquals(2, result.split("\n").length);

http://git-wip-us.apache.org/repos/asf/accumulo/blob/24edc84a/test/src/main/java/org/apache/accumulo/test/functional/ScannerContextIT.java
----------------------------------------------------------------------
diff --git a/test/src/main/java/org/apache/accumulo/test/functional/ScannerContextIT.java
b/test/src/main/java/org/apache/accumulo/test/functional/ScannerContextIT.java
index cc581f8..91c066f 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/ScannerContextIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/ScannerContextIT.java
@@ -67,7 +67,7 @@ public class ScannerContextIT extends AccumuloClusterHarness {
   @Before
   public void checkCluster() throws Exception {
     Assume.assumeThat(getClusterType(), CoreMatchers.is(ClusterType.MINI));
-    MiniAccumuloClusterImpl mac = (MiniAccumuloClusterImpl) getCluster();
+    MiniAccumuloClusterImpl.class.cast(getCluster());
     fs = FileSystem.get(CachedConfiguration.getInstance());
   }
 


Mime
View raw message