tkobayas commented on PR #6313:
URL: 
https://github.com/apache/incubator-kie-drools/pull/6313#issuecomment-2817877435

   > Sorry I was thinking to the situation where an event is inserted but the 
firing, and then the evaluation if the event matches at least one rule or not, 
is delayed. No for sure we cannot apply this PR as it is and at the moment I 
cannot think to any other way to do this.
   
   If we introduce a method like `ksession.insertAndFireAllRules`, we can pass 
a flag to `PropagationEntry.Insert` to decide whether delay scheduling or not. 
But it would require many changes, so it may not be worth doing.
   
   > On a side note, and slightly unrelated to this, shouldn't we remove the 
scheduled expiration job here 
https://github.com/apache/incubator-kie-drools/pull/6313/files#diff-fa0b75cf8efc9dfa36a928295fae390ee65153416cf930306c71885a3f3df025R234
 in a similar way as you did in the Ansible integration?
   
   Yes, I filed a fix-only PR. 
https://github.com/apache/incubator-kie-drools/pull/6317 


-- 
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]

Reply via email to