feat(medusa): Bulk emit events

[adrien2p]

What
I can see during the perf tests that inserting lots of events create a bottleneck which is resolve by the bulk emit. During the refactoring we happen a lot to have the ability to bulk emit instead of having to emit them one by one. The perf are improved a lot

FIXES CORE-1220

[changeset-bot]

🦋 Changeset detected

Latest commit: 761745e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@medusajs/inventory	Patch
medusa-test-utils	Patch
@medusajs/medusa	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated
medusa-docs	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 13, 2023 at 0:34AM (UTC)
staging	✅ Ready (Inspect)	Visit Preview	💬 Add your feedback	Mar 13, 2023 at 0:34AM (UTC)

[olivermrbl]

I like this! Would be great to add some tests to the PR :)

[olivermrbl]

@adrien2p Will you investigate the failing check? 🙏

[adrien2p]

@adrien2p Will you investigate the failing check? 🙏

Sure 👌
what db is used in the CI? I ll look at it. But it seams that returning * is only supported by postgres, maria and MSSql, ill find another solution

[olivermrbl]

But it seams that returning * is only supported by postgres, maria and MSSql, ill find another solution

Got it - that was what we were dreading.

We are testing against a SQLite DB in the pipeline as well, so that's why it's failing.

[adrien2p]

But it seams that returning * is only supported by postgres, maria and MSSql, ill find another solution

Got it - that was what we were dreading.

We are testing against a SQLite DB in the pipeline as well, so that's why it's failing.

It seams that it should be available in sqlite from v3.35 https://www.sqlite.org/lang_returning.html i wonder which version are we using

[adrien2p]

@olivermrbl I have opened an issue here in the mean time I ll do a fix

[olivermrbl]

Is it worth spending some time looking into how/if we could use the addBulk in the enqueuer as well? Otherwise, most emit calls will not benefit from these optimizations, as they are part of a transaction i.e. the staged job flow.

[adrien2p]

Is it worth spending some time looking into how/if we could use the addBulk in the enqueuer as well? Otherwise, most emit calls will not benefit from these optimizations, as they are part of a transaction i.e. the staged job flow.

Yes I will do that it is a very good point, though, the ones in the transaction benefit from the bulk insert of the staged jobs

[adrien2p]

@olivermrbl I have updated the enqueuer, let me know wdyt

[olivermrbl]

Merge as you please. Afterward (or now), I'll publish a canary, and we can run tests in a multi-process env before we release. Okay?

[adrien2p]

/snapshot-this

[github-actions]

🚀 A snapshot release has been made for this PR

Test the snapshots by updating your package.json with the newly published versions:

yarn add @medusajs/[email protected]

yarn add @medusajs/[email protected]

yarn add [email protected]

Latest commit: 601d20e