Avoid creating an ObjectMapper for each set/getItemValue

Signed-off-by: Niels Thykier <[email protected]>
dcsaorg · Nov 21, 2023 · aa50cb5 · aa50cb5
1 parent faf5f01
commit aa50cb5
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/core/src/main/java/org/dcsa/conformance/core/state/MemorySortedPartitionsNonLockingMap.java b/core/src/main/java/org/dcsa/conformance/core/state/MemorySortedPartitionsNonLockingMap.java
@@ -9,22 +9,25 @@
 import java.util.TreeMap;
 
 public class MemorySortedPartitionsNonLockingMap implements SortedPartitionsNonLockingMap {
+  private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper();
+
   private final HashMap<String, TreeMap<String, JsonNode>> memoryMap = new HashMap<>();
 
+
   @SneakyThrows
   @Override
   public synchronized void setItemValue(String partitionKey, String sortKey, JsonNode value) {
-    JsonNode valueCopy = new ObjectMapper().readTree(value.toString());
+    JsonNode valueCopy = OBJECT_MAPPER.readTree(value.toString());
     memoryMap
         .computeIfAbsent(partitionKey, (ignoredKey) -> new TreeMap<>())
-        .put(sortKey, new ObjectMapper().createObjectNode().set("value", valueCopy));
+        .put(sortKey, OBJECT_MAPPER.createObjectNode().set("value", valueCopy));
   }
 
   @Override
   public synchronized JsonNode getItemValue(String partitionKey, String sortKey) {
     return memoryMap
         .getOrDefault(partitionKey, new TreeMap<>())
-        .getOrDefault(sortKey, new ObjectMapper().createObjectNode())
+        .getOrDefault(sortKey, OBJECT_MAPPER.createObjectNode())
         .get("value");
   }