Remove unneed deep copies

Signed-off-by: Peter Nied <peternied@hotmail.com>
peternied · Mar 7, 2024 · d2843cf · d2843cf
1 parent cafb776
commit d2843cf
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/org/opensearch/security/dlic/rest/api/ConfigUpgradeApiAction.java b/src/main/java/org/opensearch/security/dlic/rest/api/ConfigUpgradeApiAction.java
@@ -107,7 +107,7 @@ void handleCanUpgrade(final RestChannel channel, final RestRequest request, fina
                 final var differencesList = new ArrayList<ValidationResult<Tuple<CType, JsonNode>>>();
                 for (final var configuration : configurations) {
                     differencesList.add(computeDifferenceToUpdate(configuration)
-                        .map(differences -> ValidationResult.success(new Tuple<CType, JsonNode>(configuration, differences.deepCopy()))));
+                        .map(differences -> ValidationResult.success(new Tuple<CType, JsonNode>(configuration, differences))));
                 }
                 return ValidationResult.combine(differencesList);
             }))
@@ -166,11 +166,11 @@ private JsonNode filterRemoveOperations(final JsonNode diff) {
         final ArrayNode filteredDiff = JsonNodeFactory.instance.arrayNode();
         diff.forEach(node -> {
             if (!isRemoveOperation(node)) {
-                filteredDiff.add(node.deepCopy());
+                filteredDiff.add(node);
                 return;
             } else {
                 if (!hasRootLevelPath(node)) {
-                    filteredDiff.add(node.deepCopy());
+                    filteredDiff.add(node);
                 }
             }
         });