samuel-beniamin opened a new issue, #1478: URL: https://github.com/apache/incubator-kie-issues/issues/1478
Currently one of the responsibilities of the `DMNCompilerImpl` is to call all registered `AfterProcessDrgElements` after each model's evaluation. This behaviour is leading to some issues, since it doesn't validate of the DRG (Decision requirements graph) Compiler would `accept` the implementation or not? Or even if the current model is in fact the model that registered the callback at all. Instead it assumes two things, first, that the `AfterProcessDrgElements` or the `DRGElementCompiler` would check if the model is accepted by that DMN compiler, second, that the call back will be idempotent as well. For the first assumption, my suggestion would be to actually check if the `DRGElementCompiler` accepts the model. For the second assumption we could do one of two things: 1. Document the `AfterProcessDrgElements` to mention it must be ready to accept many calls that are not related. And/Or, 2. `DMNCompilerImpl` should register the callback to the model that is related to it, and only call the callback on a valid and related model. The current behaviour as is, was causing an issue that was partially addressed in a PR fixing [incubator-kie-issues#1474](https://github.com/apache/incubator-kie-drools/pull/6080). -- 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]
