Jackson should use HybridJacksonPool

quarkusio · Mar 27, 2024 · f361170 · f361170
1 parent aacd5c4
commit f361170
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/...ackson/deployment/src/test/java/io/quarkus/jackson/deployment/JacksonDefaultPoolTest.java b/...ackson/deployment/src/test/java/io/quarkus/jackson/deployment/JacksonDefaultPoolTest.java
@@ -0,0 +1,14 @@
+package io.quarkus.jackson.deployment;
+
+import org.assertj.core.api.Assertions;
+import org.junit.jupiter.api.Test;
+
+import com.fasterxml.jackson.core.util.JsonRecyclerPools;
+
+public class JacksonDefaultPoolTest {
+
+    @Test
+    public void validateDefaultJacksonPool() {
+        Assertions.assertThat(JsonRecyclerPools.defaultPool()).isInstanceOf(JsonRecyclerPools.LockFreePool.class);
+    }
+}
diff --git a/...ntime/src/main/java/io/quarkus/jackson/runtime/VertxHybridPoolObjectMapperCustomizer.java b/...ntime/src/main/java/io/quarkus/jackson/runtime/VertxHybridPoolObjectMapperCustomizer.java
@@ -10,7 +10,9 @@ public class VertxHybridPoolObjectMapperCustomizer implements ObjectMapperCustom
 
     @Override
     public void customize(ObjectMapper objectMapper) {
-        if (objectMapper.getFactory()._getRecyclerPool() == JsonRecyclerPools.defaultPool()) {
+        var existingMapperPool = objectMapper.getFactory()._getRecyclerPool();
+        // JsonRecyclerPools.defaultPool() by default should create a LockFreePool
+        if (!(existingMapperPool instanceof JsonRecyclerPools.LockFreePool)) {
             objectMapper.getFactory().setRecyclerPool(HybridJacksonPool.getInstance());
         }
     }