Modernize

5 files changed, 19 insertions, 17 deletions

diff --git a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/MissingUnitException.java b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/MissingUnitException.java
index 838e13fc4ee..58a862f4878 100644
--- a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/MissingUnitException.java
+++ b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/MissingUnitException.java
@@ -1,19 +1,21 @@
 // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.clustercontroller.utils.staterestapi.errors;
 
+import java.util.List;
+
 public class MissingUnitException extends StateRestApiException {
 
-    private static String createMessage(String[] path, int level) {
+    private static String createMessage(List<String> path, int level) {
         StringBuilder sb = new StringBuilder();
         sb.append("No such resource '");
         for (int i=0; i<=level; ++i) {
             if (i != 0) sb.append('/');
-            sb.append(path[i]);
+            sb.append(path.get(i));
         }
         return sb.append("'.").toString();
     }
 
-    public MissingUnitException(String[] path, int level) {
+    public MissingUnitException(List<String> path, int level) {
         super(createMessage(path, level));
         setHtmlCode(404);
         setHtmlStatus(getMessage());
diff --git a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/OperationNotSupportedForUnitException.java b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/OperationNotSupportedForUnitException.java
index 342f568eacc..abc55d68bc6 100644
--- a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/OperationNotSupportedForUnitException.java
+++ b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/errors/OperationNotSupportedForUnitException.java
@@ -2,16 +2,17 @@
 package com.yahoo.vespa.clustercontroller.utils.staterestapi.errors;
 
 import java.util.Arrays;
+import java.util.List;
 
 public class OperationNotSupportedForUnitException extends StateRestApiException {
 
-    private static String createMessage(String[] path, String description) {
+    private static String createMessage(List<String> path, String description) {
         return new StringBuilder()
-                .append(Arrays.toString(path)).append(": ").append(description)
+                .append(Arrays.toString(path.toArray())).append(": ").append(description)
                 .toString();
     }
 
-    public OperationNotSupportedForUnitException(String path[], String description) {
+    public OperationNotSupportedForUnitException(List<String> path, String description) {
         super(createMessage(path, description));
         setHtmlCode(405);
         setHtmlStatus("Operation not supported for resource");
diff --git a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/requests/UnitRequest.java b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/requests/UnitRequest.java
index 53abfff8aba..52bef3b09c0 100644
--- a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/requests/UnitRequest.java
+++ b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/requests/UnitRequest.java
@@ -1,8 +1,10 @@
 // Copyright Yahoo. Licensed under the terms of the Apache 2.0 license. See LICENSE in the project root.
 package com.yahoo.vespa.clustercontroller.utils.staterestapi.requests;
 
+import java.util.List;
+
 public interface UnitRequest {
 
-    String[] getUnitPath();
+    List<String> getUnitPath();
 
 }
diff --git a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/response/SetResponse.java b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/response/SetResponse.java
index 1c704ea3b63..2287abb5ca7 100644
--- a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/response/SetResponse.java
+++ b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/response/SetResponse.java
@@ -24,7 +24,7 @@ public class SetResponse {
     public boolean getWasModified() { return wasModified; }
 
     /**
-     * Human readable reason.
+     * Human-readable reason.
      *
      * @return reason as string
      */
diff --git a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/server/RestApiHandler.java b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/server/RestApiHandler.java
index 654481aee33..ceec4f67e1b 100644
--- a/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/server/RestApiHandler.java
+++ b/clustercontroller-utils/src/main/java/com/yahoo/vespa/clustercontroller/utils/staterestapi/server/RestApiHandler.java
@@ -25,7 +25,6 @@ import java.time.Clock;
 import java.time.Duration;
 import java.time.Instant;
 import java.util.ArrayList;
-import java.util.Arrays;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -64,7 +63,7 @@ public class RestApiHandler implements HttpRequestHandler {
         Instant start = clock.instant();
 
         try{
-            final String[] unitPath = createUnitPath(request);
+            List<String> unitPath = createUnitPath(request);
             if (request.getHttpOperation().equals(HttpRequest.HttpOp.GET)) {
                 final int recursiveLevel = getRecursiveLevel(request);
                 UnitResponse data = restApi.getState(new UnitStateRequest() {
@@ -73,9 +72,7 @@ public class RestApiHandler implements HttpRequestHandler {
                         return recursiveLevel;
                     }
                     @Override
-                    public String[] getUnitPath() {
-                        return unitPath;
-                    }
+                    public List<String> getUnitPath() { return unitPath; }
                 });
                 return new JsonHttpResult().setJson(jsonWriter.createJson(data));
             } else {
@@ -87,7 +84,7 @@ public class RestApiHandler implements HttpRequestHandler {
                         return setRequestData.stateMap;
                     }
                     @Override
-                    public String[] getUnitPath() {
+                    public List<String> getUnitPath() {
                         return unitPath;
                     }
                     @Override
@@ -137,9 +134,9 @@ public class RestApiHandler implements HttpRequestHandler {
         }
     }
 
-    private String[] createUnitPath(HttpRequest request) {
-        List<String> path = Arrays.asList(request.getPath().split("/"));
-        return path.subList(3, path.size()).toArray(new String[0]);
+    private List<String> createUnitPath(HttpRequest request) {
+        List<String> path = List.of(request.getPath().split("/"));
+        return path.subList(3, path.size());
     }
 
     private int getRecursiveLevel(HttpRequest request) throws StateRestApiException {