Details

    • Type: Enhancement
    • Status: Resolved (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: dmn engine
    • Labels:
      None

      Description

      Currently, DMN layer Evaluator method:

      org.kie.dmn.core.ast.DMNDTExpressionEvaluator.evaluate(org.kie.dmn.api.core.event.DMNRuntimeEventManager, org.kie.dmn.api.core.DMNResult)
      

      calls down to FEEL layer function:

      org.kie.dmn.feel.runtime.functions.DTInvokerFunction.invoke(org.kie.dmn.feel.lang.EvaluationContext, java.lang.Object[ ])
      

      In order to evaluate Decision Table using DecisionTableImpl.

      Currently, the whole Decision Table FEEL expression/UT are compiled to CompiledExpression and this enable for fast performance during execution and multiple evaluation, however, the parameter binding for the invocation from the DMN layer does not leverage FEEL expression compilation of the parameters ( information/knowledge requirements coming from the DRG graph ) hence the parameter binding is not really as fast as it could be across several evaluation.

      The current proposal leverage compilation for the parameter (names) which are actually FEEL expression, in order to move from this scenario:

      to this:

      Conclusion the current proposal optimize the DMN parameter binding for the Decision Table invocation across several evaluations by at least an order of magnitude*

      * if compared to baseline, using a JUnit test part of the actual code base as benchmark

        Gliffy Diagrams

          Attachments

          1. AFTER.png
            AFTER.png
            64 kB
          2. BEFORE.png
            BEFORE.png
            65 kB

            Activity

              People

              • Assignee:
                tari_manga Matteo Mortari
                Reporter:
                tari_manga Matteo Mortari
                Tester:
                Tibor Zimanyi
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: