Remove ObjectMapper from BaseYangOpenApiGenerator 29/106429/3
authorIvan Hrasko <ivan.hrasko@pantheon.tech>
Thu, 8 Jun 2023 13:15:51 +0000 (15:15 +0200)
committerRobert Varga <nite@hq.sk>
Mon, 12 Jun 2023 08:09:14 +0000 (08:09 +0000)
Its unfortunate to use heavy ObjectMapper for logging purposes.

Change-Id: I6357d4bd768065144dc8dee2d4f0b722278b5994
Signed-off-by: Ivan Hrasko <ivan.hrasko@pantheon.tech>
restconf/restconf-openapi/src/main/java/org/opendaylight/restconf/openapi/impl/BaseYangOpenApiGenerator.java

index 8edbd6f87e17108289fbb5456d17f46308310e9b..0f6688f5317a3c0dcc325801e8c31cc9a2f312a1 100644 (file)
@@ -18,8 +18,6 @@ import static org.opendaylight.restconf.openapi.model.builder.OperationBuilder.g
 import static org.opendaylight.restconf.openapi.util.RestDocgenUtil.resolvePathArgumentsName;
 
 import com.fasterxml.jackson.databind.JsonNode;
-import com.fasterxml.jackson.databind.ObjectMapper;
-import com.fasterxml.jackson.databind.SerializationFeature;
 import com.fasterxml.jackson.databind.node.ArrayNode;
 import com.fasterxml.jackson.databind.node.JsonNodeFactory;
 import com.fasterxml.jackson.databind.node.ObjectNode;
@@ -71,7 +69,6 @@ public abstract class BaseYangOpenApiGenerator {
 
     private static final String API_VERSION = "1.0.0";
     private static final String OPEN_API_VERSION = "3.0.3";
-    private static final ObjectMapper MAPPER = new ObjectMapper();
 
     private final DefinitionGenerator jsonConverter = new DefinitionGenerator();
     private final DOMSchemaService schemaService;
@@ -84,10 +81,6 @@ public abstract class BaseYangOpenApiGenerator {
     private static final ArrayNode SECURITY = JsonNodeFactory.instance.arrayNode()
             .add(JsonNodeFactory.instance.objectNode().set("basicAuth", JsonNodeFactory.instance.arrayNode()));
 
-    static {
-        MAPPER.configure(SerializationFeature.INDENT_OUTPUT, true);
-    }
-
     protected BaseYangOpenApiGenerator(final Optional<DOMSchemaService> schemaService) {
         this.schemaService = schemaService.orElse(null);
     }
@@ -222,9 +215,6 @@ public abstract class BaseYangOpenApiGenerator {
             final Map<String, Schema> schemas = jsonConverter.convertToSchemas(module, schemaContext,
                 definitionNames, isForSingleModule);
             docBuilder.getComponents().schemas().putAll(schemas);
-            if (LOG.isDebugEnabled()) {
-                LOG.debug("Document: {}", MAPPER.writeValueAsString(docBuilder.build()));
-            }
         } catch (final IOException e) {
             LOG.error("Exception occurred in DefinitionGenerator", e);
         }