Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improved: Add delegator method to create in mass many entities (OFBIZ-13176) #857

Conversation

nmalin
Copy link
Contributor

@nmalin nmalin commented Nov 14, 2024

When you need to load huge record on your database (like system synchronization, data import) OFBiz can have latency with the delegator.create. Incriminated unitary workflow and eeca control. You can disable eeca on use a dedicate delegator but sometime it's not enough.

So for special case, we implement a new function delegator.createAllByBatchProcess that will use an insert with one transaction through the standard sql method INSERT TO ... VALUES ... .

By the way, like the process isn't unitary, the eeca system can't run on this batch, so this function must therefore be used with care

Thanks to Néréide team for this contribution

nmalin and others added 2 commits November 14, 2024 17:35
…-13176)

When you need to load huge record on your database (like system synchronization, data import) OFBiz can have latency with the delegator.create.
Incriminated unitary workflow and eeca control. You can disable eeca on use a dedicate delegator but sometime it's not enough.

So for special case, we implement a new function delegator.createAllByBatchProcess that will use an insert with one transaction through the standard sql method INSERT TO ... VALUES ... .

By the way, like the process isn't unitary, the eeca system can't run on this batch, so this function must therefore be used with care

Thanks to Néréide team for this contribution
Copy link

sonarcloud bot commented Nov 15, 2024

@gilPts gilPts merged commit d7a1b40 into apache:trunk Nov 15, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants