This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo-proxy.git


The following commit(s) were added to refs/heads/main by this push:
     new 4155a74  Fix some usages of deprecation (#61)
4155a74 is described below

commit 4155a7418bcecc52b91bfef7e5e67120dd26e427
Author: Dom G <domgargu...@apache.org>
AuthorDate: Tue Jan 17 16:50:15 2023 -0500

    Fix some usages of deprecation (#61)
---
 src/main/java/org/apache/accumulo/proxy/Proxy.java       | 2 +-
 src/main/java/org/apache/accumulo/proxy/ProxyServer.java | 9 +++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/src/main/java/org/apache/accumulo/proxy/Proxy.java 
b/src/main/java/org/apache/accumulo/proxy/Proxy.java
index da31732..26e6dec 100644
--- a/src/main/java/org/apache/accumulo/proxy/Proxy.java
+++ b/src/main/java/org/apache/accumulo/proxy/Proxy.java
@@ -159,7 +159,7 @@ public class Proxy implements KeywordExecutable {
         .forName(
             proxyProps.getProperty("protocolFactory", 
TCompactProtocol.Factory.class.getName()))
         .asSubclass(TProtocolFactory.class);
-    TProtocolFactory protoFactory = protoFactoryClass.newInstance();
+    TProtocolFactory protoFactory = 
protoFactoryClass.getDeclaredConstructor().newInstance();
     int port = Integer.parseInt(proxyProps.getProperty("port"));
     String hostname = proxyProps.getProperty(THRIFT_SERVER_HOSTNAME,
         THRIFT_SERVER_HOSTNAME_DEFAULT);
diff --git a/src/main/java/org/apache/accumulo/proxy/ProxyServer.java 
b/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
index e21b990..92e363d 100644
--- a/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
+++ b/src/main/java/org/apache/accumulo/proxy/ProxyServer.java
@@ -18,6 +18,7 @@ package org.apache.accumulo.proxy;
 
 import static java.nio.charset.StandardCharsets.UTF_8;
 
+import java.lang.reflect.InvocationTargetException;
 import java.nio.ByteBuffer;
 import java.util.ArrayList;
 import java.util.Collection;
@@ -802,7 +803,7 @@ public class ProxyServer implements AccumuloProxy.Iface {
         pscan.type = ScanType.valueOf(scan.getType().toString());
         pscan.state = ScanState.valueOf(scan.getState().toString());
         TabletId e = scan.getTablet();
-        pscan.extent = new 
org.apache.accumulo.proxy.thrift.KeyExtent(e.getTableId().toString(),
+        pscan.extent = new 
org.apache.accumulo.proxy.thrift.KeyExtent(e.getTable().toString(),
             TextUtil.getByteBuffer(e.getEndRow()), 
TextUtil.getByteBuffer(e.getPrevEndRow()));
         pscan.columns = new ArrayList<>();
         if (scan.getColumns() != null) {
@@ -856,7 +857,7 @@ public class ProxyServer implements AccumuloProxy.Iface {
         pcomp.entriesRead = comp.getEntriesRead();
         pcomp.entriesWritten = comp.getEntriesWritten();
         TabletId e = comp.getTablet();
-        pcomp.extent = new 
org.apache.accumulo.proxy.thrift.KeyExtent(e.getTableId().toString(),
+        pcomp.extent = new 
org.apache.accumulo.proxy.thrift.KeyExtent(e.getTable().toString(),
             TextUtil.getByteBuffer(e.getEndRow()), 
TextUtil.getByteBuffer(e.getPrevEndRow()));
         pcomp.inputFiles = new ArrayList<>();
         if (comp.getInputFiles() != null) {
@@ -2078,8 +2079,8 @@ public class ProxyServer implements AccumuloProxy.Iface {
     props.putAllStrings(properties);
     AuthenticationToken token;
     try {
-      token = tokenClass.newInstance();
-    } catch (InstantiationException | IllegalAccessException e) {
+      token = tokenClass.getDeclaredConstructor().newInstance();
+    } catch (ReflectiveOperationException e) {
       logger.error("Error constructing authentication token", e);
       throw new AccumuloException(e);
     }

Reply via email to