The JacksonCodec implementation of toString/toBuffer lacks of efficie…

…ncy compared to the DatabindCodec implementation which uses the buffer recyler of Jackson.

Update the JacksonCodec implementation to use the buffer recycler instead.

src/main/java/io/vertx/core/json/jackson/JacksonCodec.java

@@ -12,11 +12,11 @@
 package io.vertx.core.json.jackson;
 
 import com.fasterxml.jackson.core.*;
+import com.fasterxml.jackson.core.io.SegmentedStringWriter;
 import com.fasterxml.jackson.core.type.TypeReference;
-import io.netty.buffer.ByteBuf;
+import com.fasterxml.jackson.core.util.BufferRecycler;
+import com.fasterxml.jackson.core.util.ByteArrayBuilder;
 import io.netty.buffer.ByteBufInputStream;
-import io.netty.buffer.ByteBufOutputStream;
-import io.netty.buffer.Unpooled;
 import io.vertx.core.buffer.Buffer;
 import io.vertx.core.json.DecodeException;
 import io.vertx.core.json.EncodeException;
@@ -28,7 +28,6 @@
 import java.io.IOException;
 import java.io.InputStream;
 import java.io.OutputStream;
-import java.io.StringWriter;
 import java.io.Writer;
 import java.lang.reflect.Method;
 import java.lang.reflect.ParameterizedType;
@@ -108,35 +107,33 @@ public <T> T fromValue(Object json, TypeReference<T> type) {
 
   @Override
   public String toString(Object object, boolean pretty) throws EncodeException {
-    StringWriter sw = new StringWriter();
-    JsonGenerator generator = createGenerator(sw, pretty);
-    try {
+    BufferRecycler br = factory._getBufferRecycler();
+    try (SegmentedStringWriter sw = new SegmentedStringWriter(br)) {
+      JsonGenerator generator = createGenerator(sw, pretty);
       encodeJson(object, generator);
-      generator.flush();
-      return sw.toString();
+      generator.close();
+      return sw.getAndClear();
     } catch (IOException e) {
       throw new EncodeException(e.getMessage(), e);
     } finally {
-      close(generator);
+      br.releaseToPool();
     }
   }
 
   @Override
   public Buffer toBuffer(Object object, boolean pretty) throws EncodeException {
-    ByteBuf buf = Unpooled.buffer();
-    // There is no need to use a try with resources here as jackson
-    // is a well-behaved and always calls the closes all streams in the
-    // "finally" block bellow.
-    ByteBufOutputStream out = new ByteBufOutputStream(buf);
-    JsonGenerator generator = createGenerator(out, pretty);
-    try {
+    BufferRecycler br = factory._getBufferRecycler();
+    try (ByteArrayBuilder bb = new ByteArrayBuilder(br)) {
+      JsonGenerator generator = createGenerator(bb, pretty);
       encodeJson(object, generator);
-      generator.flush();
-      return Buffer.buffer(buf);
-    } catch (IOException e) {
+      generator.close();
+      byte[] result = bb.toByteArray();
+      bb.release();
+      return Buffer.buffer(result);
+    } catch (java.io.IOException e) {
       throw new EncodeException(e.getMessage(), e);
     } finally {
-      close(generator);
+      br.releaseToPool();
     }
   }
 
@@ -286,7 +283,7 @@ static void close(Closeable parser) {
   }
 
   // In recursive calls, the callee is in charge of opening and closing the data structure
-  private static void encodeJson(Object json, JsonGenerator generator) throws EncodeException {
+  public static void encodeJson(Object json, JsonGenerator generator) throws EncodeException {
     try {
       if (json instanceof JsonObject) {
         json = ((JsonObject)json).getMap();