Fix usage of apis deprecated from Java 9.

+ rewrite if to switch-case.

2 files changed, 26 insertions, 16 deletions

diff --git a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
index 619643a32b6..db2ed6b0b83 100644
--- a/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
+++ b/logserver/src/main/java/com/yahoo/logserver/AbstractPluginLoader.java
@@ -5,6 +5,7 @@ import com.yahoo.log.LogLevel;
 import com.yahoo.plugin.Plugin;
 import com.yahoo.plugin.SystemPropertyConfig;
 
+import java.lang.reflect.InvocationTargetException;
 import java.util.logging.Logger;
 
 /**
@@ -20,8 +21,8 @@ public abstract class AbstractPluginLoader implements PluginLoader {
     protected void loadFromClass(Class<? extends Plugin> pluginClass) {
         Plugin plugin;
         try {
-            plugin = (Plugin) pluginClass.newInstance();
-        } catch (InstantiationException | IllegalAccessException e) {
+            plugin = pluginClass.getDeclaredConstructor().newInstance();
+        } catch (InstantiationException | IllegalAccessException | NoSuchMethodException | InvocationTargetException e) {
             log.log(LogLevel.ERROR, pluginClass.getName() + ": load failed: " + e);
             throw new RuntimeException(e);
         }
diff --git a/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java b/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
index 2e0b3d67c49..434131878c0 100644
--- a/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
+++ b/logserver/src/test/java/com/yahoo/logserver/handlers/logmetrics/test/LogMetricsTestCase.java
@@ -83,20 +83,29 @@ public class LogMetricsTestCase {
         assertEquals(levelCount.entrySet().size(), 5);
         for (Map.Entry<String, Long> entry : levelCount.entrySet()) {
             String key = entry.getKey();
-            if (key.equals("config")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("info")) {
-                assertEquals(entry.getValue(), new Long(4));
-            } else if (key.equals("warning")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("severe")) {
-                assertEquals(entry.getValue(), new Long(0));
-            } else if (key.equals("error")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("fatal")) {
-                assertEquals(entry.getValue(), new Long(1));
-            } else if (key.equals("debug")) {
-                assertEquals(entry.getValue(), new Long(0));  // always 0
+            switch (key) {
+                case "config":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "info":
+                    assertEquals(entry.getValue(), Long.valueOf(4));
+                    break;
+                case "warning":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "severe":
+                    assertEquals(entry.getValue(), Long.valueOf(0));
+                    break;
+                case "error":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "fatal":
+                    assertEquals(entry.getValue(), Long.valueOf(1));
+                    break;
+                case "debug":
+                    assertEquals(entry.getValue(), Long.valueOf(0));  // always 0
+
+                    break;
             }
         }
         a.close();