Module cannot be installed with large user base

Indranil Roy → made their first commit to this issue’s fork.

Hi @3li, I'm working on this issue.

Hi I have raised the MR. Needs review.

Thanks, couple of things based on the diff.

So working with a batch is great, but the way it is implemented, all users are still loaded in one call. We need to initialise a sandbox inside the batch context, load all user id's there using just a entity query on the user storage and then let batch iterate over them (either per user or in batches of lets say 100 users) until we are finished.

Setting up the batch can also be done using the BatchBuilder class.

The methods of the new class don't use dependency injection. So either solve that, or move the batch handling into the existing service and just add the dependency injection for the messenger service there.

Thanks @kriboogh for your suggestion, will surely work on this.

@kriboogh worked on your feedbacks

@rajdip_755 i have solved the issue

@kalash-j I have applied the updated MR as a patch in a drupal instance with 2256 numbers of users. It's working fine for me attaching below the screenshot of the installation process of the module using drush command and the UI.

Thanks !

Looks ok for me.

@kriboogh is there any work left on this issue , or can you change the status of the ticket?

Automatically closed - issue fixed for 2 weeks with no activity.