chore: Improve groups.create endpoint for large amounts of members

[sampaiodiego]

Proposed changes (including videos or screenshots)

Some key changes:

• changed the signature of createRoom function, to receive an IUser instead of a username. This was to prevent multiple trips to DB being done to fetch data from the same user
• change how subscriptions are being created, now it creates an array of all subscriptions that need to be created and create them all at once using insertMany. Before they were being created synchronously one by one
• update members count in the room object in a single command instead of single increments for each user
• update user's __rooms property with a single updateMany command instead of multiple updateOne for each user/member provided

You can see below a load test comparing the old and the new code.

This test was performed by calling the endpoint providing an array with 100 members during 2 minutes by 5 simultaneous virtual users:

This one was also provided an array with 100 members but during 5 minutes by 5 simultaneous virtual users. another difference is that the user calling the API was already a member of 70k+ rooms:

we can see in both cases an improvement of around 50% either in number of requests or response time.

I still plan to add some tests.. I noticed we don't have tests covering even common use cases.

Issue(s)

Steps to test or reproduce

Further comments

[changeset-bot]

⚠️ No Changeset found

Latest commit: a524c2a

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

[codecov]

Codecov Report

Merging #30499 (a524c2a) into develop (9aee190) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #30499      +/-   ##
===========================================
- Coverage    50.87%   50.81%   -0.06%     
===========================================
  Files          809      803       -6     
  Lines        15020    15014       -6     
  Branches      2729     2733       +4     
===========================================
- Hits          7642     7630      -12     
+ Misses        6969     6959      -10     
- Partials       409      425      +16     

Flag	Coverage Δ
e2e	48.54% <ø> (-0.02%)	⬇️
unit	62.56% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.