You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by ct...@apache.org on 2015/12/10 16:42:07 UTC
[2/2] accumulo git commit: ACCUMULO-3948 Cleanup unused variables
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 <ct...@apache.org>
Authored: Thu Dec 10 10:17:17 2015 -0500
Committer: Christopher Tubbs <ct...@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());
}