Uploaded image for project: 'Kogito'
  1. Kogito
  2. KOGITO-5368

Unify Json merge implementation

XMLWordPrintable

    • False
    • False
    • Undefined
    • 2021 Week 34-36 (from Aug 23), 2021 Week 37-39 (from Sep 13), 2021 Week 40-42 (from Oct 4)

      Currently we have two different implementations of mergeJson

      The one being used in RestWorkItemHandler
      https://github.com/kiegroup/kogito-runtimes/blob/9bd1c5ccc3c5d297e44d2001286de0befd81e0d7/kogito-workitems/kogito-rest-workitem/src/main/java/org/kogito/workitem/rest/RestWorkItemHandlerUtils.java

      The one being used in OpenAPIHandler
      https://github.com/kiegroup/kogito-runtimes/blob/master/kogito-workitems/kogito-openapi-workitem/src/main/java/org/kogito/workitem/openapi/JsonNodeResultHandler.java#L41

      We should move one of the two to a common static class and invoke it from both handlers.
      In order to chose one of the two, I think we need some performance metrics

              ftirados Francisco Javier Tirado Sarti
              ftirados Francisco Javier Tirado Sarti
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Created:
                Updated:
                Resolved: