MINOR: Improve PeriodicBatchEntityTrigger Workflows performance

IceS2 commented 6 days ago

In order to Increase the performance of the Flowable Workflows (Mainly for the PeriodicBatchEntityTrigger), we are:

Updating the default Async Executor Flowable configurations to handle more workload The default configurations are pretty bad for an asset intensive production environment and thus we are updating them.
Refactoring how we call the MainWorkflow from the TriggerWorkflow We are now relying on CallActivity to call the MainWorkflow instead of doing it programatically through a JavaDelegate on a ServiceTask.

This is pretty important for the PeriodicBatchEntityTrigger, where we are now relying on MultiInstanceLoopCharacteristics for the concurrent task execution instead of dealing with it manually.

It is a BPMN2.0 natural way of solving this issues that I didn't realized it would work for a whole workflow before.

As a plus, it also takes care of waiting for the MainWorkflow to finish before finishing the TriggerWorkflow.
Small fix to avoid patching an Entity if the updated Certification would be the same as the one they currently have.

#

#

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

sonarcloud[bot] commented 5 days ago

Quality Gate passed for 'open-metadata-ingestion'

IceS2 commented 4 days ago