gitgabrio commented on issue #73: URL: https://github.com/apache/incubator-kie-issues/issues/73#issuecomment-2098010638
Based on analysys, I found that FEEL usage itself is not duplicated, i.e. both code paths ultimately invoke `FEELFunction#invokeReflectively(EvaluationContext, Object[])}` The difference is the overall "context" and the way messages/errors are dealt with: 1. in the intepreted way, the `MsgUtil.report` is used in case of errors, and the returned `EvaluatorResultImpl` has a flag to indicate a failure 2. in the compiled way, the `EvaluationContext.notifyEvent()` is invoked in case of errors, and the returned object is the result of function evaluation itself. Merging the two paths seems pretty error prone and anyway the core FEEL implementation itself seems not duplicated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
