[v3.1.0] RulesEngine does not update the compiled rules atomically.

Starting this thread to discuss the synchronization issue due to the compiled rule not updated atomically.

Current Workaround

We need to perform:

1. RulesEngine.RemoveWorkflow(string workflowName) and
2. RulesEngine.AddWorkflow(WorkflowRules[] workflowRules) in sequence to clear the compiled rules (_compileRules) in RulesCache.cs.
3. The compiled rules are then refreshed when we call RulesEngine.ExecuteAllRulesAsync(string workflowName, params RuleParameter[] ruleParams)(see [1], [2], [3]).

Synchronization Issue

Caller needs to synchronize the incoming requests for a worflow name by SemaphoreSlim lock (or equivalent) when they need to execute rules via the above three operations otherwise workflow rules may not be executed say if one of the requests is trying to ExecuteAllRulesAsync while the other request is RulesEngine.RemoveWorkflow.

Proposed

Provide an atomic way to update the (_compileRules) in RulesCache.cs by a new method RulesEngine.AddOrUpdateWorkflowRules(WorkflowRules[] workflowRules, params Type[] type) .

[abbasc52]

this has been fixed in v3.2.0

[abbasc52]

We don’t have any strict cadence, as for this new change, we should be good to release by 9th or 10th June