Scalability

[beck24]

I haven't tested this with a large recipient base, but just scanning the code it looks like the recipients are retrieved from the DB in a single egef* operation, which will likely pose problems with large numbers of recipients.

Additionally, what's the rationale behind using exec() to trigger the sending? I always get nervous with exec calls :)

[beck24]

#21

Perhaps the php path should be a plugin setting if exec is required

[jeabakker]

I haven't tested this with a large recipient base, but just scanning the code it looks like the recipients are retrieved from the DB in a single egef* operation, which will likely pose problems with large numbers of recipients.

The recipients are fetched from the DB but they aren't made into full entities just array(guid, email) this should help with OOM

Additionally, what's the rationale behind using exec() to trigger the sending? I always get nervous with exec calls :)

Offloading the load out of the cron process, so sending the newsletter to 1000+ users doesn't make the cron wait until the sending is complete. This way it doesn't affect other cron tasks (all that much)