accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [15/21] accumulo git commit: ACCUMULO-3820 Remove unused variable
Date Wed, 20 May 2015 16:24:59 GMT
ACCUMULO-3820 Remove unused variable

* Remove warning for unused variable.
* Update comment explaining why the parameter wasn't used.


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

Branch: refs/heads/master
Commit: 6524b0720350170696c1692430ad47dd5f4bffd0
Parents: ed5b53b
Author: Christopher Tubbs <ctubbsii@apache.org>
Authored: Wed May 20 11:08:07 2015 -0400
Committer: Christopher Tubbs <ctubbsii@apache.org>
Committed: Wed May 20 11:08:07 2015 -0400

----------------------------------------------------------------------
 .../apache/accumulo/test/randomwalk/multitable/CopyTool.java   | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/accumulo/blob/6524b072/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
----------------------------------------------------------------------
diff --git a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
index b4e509c..251a064 100644
--- a/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
+++ b/test/src/main/java/org/apache/accumulo/test/randomwalk/multitable/CopyTool.java
@@ -19,9 +19,9 @@ package org.apache.accumulo.test.randomwalk.multitable;
 import java.io.IOException;
 
 import org.apache.accumulo.core.client.ClientConfiguration;
+import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.Connector;
 import org.apache.accumulo.core.client.ZooKeeperInstance;
-import org.apache.accumulo.core.client.ClientConfiguration.ClientProperty;
 import org.apache.accumulo.core.client.admin.DelegationTokenConfig;
 import org.apache.accumulo.core.client.mapreduce.AccumuloInputFormat;
 import org.apache.accumulo.core.client.mapreduce.AccumuloOutputFormat;
@@ -74,9 +74,7 @@ public class CopyTool extends Configured implements Tool {
     final AuthenticationToken token;
     if (clientConf.getBoolean(ClientProperty.INSTANCE_RPC_SASL_ENABLED.getKey(), false))
{
       // Use the Kerberos creds to request a DelegationToken for MapReduce to use
-      final String keytab = args[1];
-
-      // Better be logged in. Could use the keytab, but we're already logged in soo..
+      // We could use the specified keytab (args[1]), but we're already logged in and don't
need to, so we can just use the current user
       KerberosToken kt = new KerberosToken();
       try {
         UserGroupInformation user = UserGroupInformation.getCurrentUser();


Mime
View raw message