From dc18e215068e6cce3048bc860739f737b2807f67 Mon Sep 17 00:00:00 2001 From: Jeffrey Manno Date: Wed, 24 Nov 2021 13:10:23 -0500 Subject: [PATCH 1/4] Fix build warnings in a few classes due to deprication --- .../accumulo/testing/performance/impl/MergeSiteConfig.java | 2 +- .../java/org/apache/accumulo/testing/randomwalk/Framework.java | 2 +- .../java/org/apache/accumulo/testing/randomwalk/Module.java | 3 ++- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/main/java/org/apache/accumulo/testing/performance/impl/MergeSiteConfig.java b/src/main/java/org/apache/accumulo/testing/performance/impl/MergeSiteConfig.java index f01afdd3..d5cfef38 100644 --- a/src/main/java/org/apache/accumulo/testing/performance/impl/MergeSiteConfig.java +++ b/src/main/java/org/apache/accumulo/testing/performance/impl/MergeSiteConfig.java @@ -32,7 +32,7 @@ public static void main(String[] args) throws Exception { Path confFile = Paths.get(args[1], "accumulo.properties"); PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class) - .newInstance(); + .getDeclaredConstructor().newInstance(); Properties props = new Properties(); diff --git a/src/main/java/org/apache/accumulo/testing/randomwalk/Framework.java b/src/main/java/org/apache/accumulo/testing/randomwalk/Framework.java index b48cfae7..a055ad14 100644 --- a/src/main/java/org/apache/accumulo/testing/randomwalk/Framework.java +++ b/src/main/java/org/apache/accumulo/testing/randomwalk/Framework.java @@ -72,7 +72,7 @@ Node getNode(String id) throws Exception { if (id.endsWith(".xml")) { node = new Module(id); } else { - node = (Test) Class.forName(id).newInstance(); + node = (Test) Class.forName(id).getDeclaredConstructor().newInstance(); } nodes.put(id, node); return node; diff --git a/src/main/java/org/apache/accumulo/testing/randomwalk/Module.java b/src/main/java/org/apache/accumulo/testing/randomwalk/Module.java index ce118dd6..0137c4ca 100644 --- a/src/main/java/org/apache/accumulo/testing/randomwalk/Module.java +++ b/src/main/java/org/apache/accumulo/testing/randomwalk/Module.java @@ -548,7 +548,8 @@ private void loadFromXml() throws Exception { nodelist = d.getDocumentElement().getElementsByTagName("fixture"); if (nodelist.getLength() > 0) { Element fixtureEl = (Element) nodelist.item(0); - fixture = (Fixture) Class.forName(getFullName(fixtureEl.getAttribute("id"))).newInstance(); + fixture = (Fixture) Class.forName(getFullName(fixtureEl.getAttribute("id"))) + .getDeclaredConstructor().newInstance(); } // parse initial node From 3d389cfabcbe01c191e83fb3425fd65b799fd72b Mon Sep 17 00:00:00 2001 From: Jeffrey Manno Date: Wed, 24 Nov 2021 14:01:19 -0500 Subject: [PATCH 2/4] one more --- .../accumulo/testing/performance/impl/PerfTestRunner.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java index c85b2673..3753d8eb 100644 --- a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java +++ b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java @@ -41,7 +41,7 @@ public static void main(String[] args) throws Exception { String accumuloVersion = args[2]; String outputDir = args[3]; - PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class) + PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class).getDeclaredConstructor() .newInstance(); AccumuloClient client = Accumulo.newClient().from(clientProps).build(); From e09a3394751710d26f783fd5ea5bba34f02e37d0 Mon Sep 17 00:00:00 2001 From: Jeffrey Manno Date: Wed, 24 Nov 2021 17:29:31 -0500 Subject: [PATCH 3/4] Update src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java Co-authored-by: Mike Miller --- .../accumulo/testing/performance/impl/PerfTestRunner.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java index 3753d8eb..56046669 100644 --- a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java +++ b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java @@ -41,7 +41,8 @@ public static void main(String[] args) throws Exception { String accumuloVersion = args[2]; String outputDir = args[3]; - PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class).getDeclaredConstructor() + PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class) + .getDeclaredConstructor().newInstance(); .newInstance(); AccumuloClient client = Accumulo.newClient().from(clientProps).build(); From cb058df059658998f363b8d0bcf7ed71104a3ce5 Mon Sep 17 00:00:00 2001 From: Jeffrey Manno Date: Wed, 24 Nov 2021 17:31:10 -0500 Subject: [PATCH 4/4] format --- .../apache/accumulo/testing/performance/impl/PerfTestRunner.java | 1 - 1 file changed, 1 deletion(-) diff --git a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java index 56046669..0a9fc0e7 100644 --- a/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java +++ b/src/main/java/org/apache/accumulo/testing/performance/impl/PerfTestRunner.java @@ -43,7 +43,6 @@ public static void main(String[] args) throws Exception { PerformanceTest perfTest = Class.forName(className).asSubclass(PerformanceTest.class) .getDeclaredConstructor().newInstance(); - .newInstance(); AccumuloClient client = Accumulo.newClient().from(clientProps).build();