For every mail send a new connection to the SMTP-server is established. When sending large amounts of mail, performance could be increased by keeping the connection open.

CommentFileSizeAuthor
smtp.mail_.inc_.patch15.25 KBsandervd

Comments

fabianx’s picture

fabianx commented
Status: Active » Needs work

It definitely looks fine to me, but might need review by the maintainers first.

However setting to NW first, because there are some open questions:

Open Questions:

* What happens if the server connection is interrupted?
* What happens if any of those mailings produce an error?

franz’s picture

franz
any
Location Montréal
commented
Issue tags: +Needs tests

This is definitely interesting. However, I see it is really about time we work on a testing framework for this module.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Issue summary: View changes

The patch needs to be rerolled too.

wundo’s picture

wundo commented
Status: Needs work » Closed (outdated)

Closing very old (dead) issues, if you think this is still relevant please re-open.