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 2018/12/04 03:28:33 UTC
[accumulo] 02/02: Fix warnings
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
commit 2b11a93f65d14fa33df5477b1cf0d2e4e32b57b7
Author: Christopher Tubbs <ct...@apache.org>
AuthorDate: Mon Dec 3 22:25:39 2018 -0500
Fix warnings
Remove unnecessary warnings suppressions
Inline deprecated import
---
.../test/java/org/apache/accumulo/hadoop/its/mapred/TokenFileIT.java | 1 -
.../java/org/apache/accumulo/hadoop/its/mapreduce/TokenFileIT.java | 1 -
proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java | 5 ++---
3 files changed, 2 insertions(+), 5 deletions(-)
diff --git a/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapred/TokenFileIT.java b/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapred/TokenFileIT.java
index 563ee4c..dbf42d0 100644
--- a/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapred/TokenFileIT.java
+++ b/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapred/TokenFileIT.java
@@ -94,7 +94,6 @@ public class TokenFileIT extends AccumuloClusterHarness {
}
- @SuppressWarnings("deprecation")
@SuppressFBWarnings(value = "PATH_TRAVERSAL_IN", justification = "path provided by test")
@Override
public int run(String[] args) throws Exception {
diff --git a/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapreduce/TokenFileIT.java b/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapreduce/TokenFileIT.java
index 9eb5be9..6837c79 100644
--- a/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapreduce/TokenFileIT.java
+++ b/hadoop-mapreduce/src/test/java/org/apache/accumulo/hadoop/its/mapreduce/TokenFileIT.java
@@ -84,7 +84,6 @@ public class TokenFileIT extends AccumuloClusterHarness {
}
}
- @SuppressWarnings("deprecation")
@SuppressFBWarnings(value = "PATH_TRAVERSAL_IN", justification = "path provided by test")
@Override
public int run(String[] args) throws Exception {
diff --git a/proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java b/proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
index 0fc83d6..fedb986 100644
--- a/proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
+++ b/proxy/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
@@ -65,7 +65,6 @@ import org.apache.accumulo.core.client.security.SecurityErrorCode;
import org.apache.accumulo.core.client.security.tokens.AuthenticationToken;
import org.apache.accumulo.core.client.security.tokens.PasswordToken;
import org.apache.accumulo.core.clientImpl.ClientConfConverter;
-import org.apache.accumulo.core.clientImpl.ConnectorImpl;
import org.apache.accumulo.core.clientImpl.Credentials;
import org.apache.accumulo.core.clientImpl.Namespace;
import org.apache.accumulo.core.clientImpl.thrift.TableOperationExceptionType;
@@ -226,8 +225,8 @@ public class ProxyServer implements AccumuloProxy.Iface {
String[] pair = ByteBufferUtil.toString(login).split(",", 2);
if (instance.getInstanceID().equals(pair[0])) {
Credentials creds = Credentials.deserialize(pair[1]);
- return ((ConnectorImpl) instance.getConnector(creds.getPrincipal(), creds.getToken()))
- .getAccumuloClient();
+ return ((org.apache.accumulo.core.clientImpl.ConnectorImpl) instance
+ .getConnector(creds.getPrincipal(), creds.getToken())).getAccumuloClient();
} else {
throw new org.apache.accumulo.core.client.AccumuloSecurityException(pair[0],
org.apache.accumulo.core.clientImpl.thrift.SecurityErrorCode.INVALID_INSTANCEID);