Comment 14 for bug 424849

Revision history for this message
Graham Binns (gmb) wrote : Re: [Bug 424849] Re: Launchpad should batch attachment notification emails

On 4 June 2010 20:10, Leo <email address hidden> wrote:
> Is it simpler to detect and batch attachments than just replacing on-
> the-fly notifications by 5-minute-interval digests?
>

At this point, yes.

The bug notification code (which can be found in
lib/canonical/launchpad/mailnotification.py [1] for anyone who wants
to take a look) is pretty ugly and is in sore need of refactoring. As
part of that work we may look at how batching works and whether we
need to do something different, but that's not the first thing on our
list of priorities. This bug is causing a lot of noise, so it's better
to fix this sooner rather than later.

 [1] http://bazaar.launchpad.net/~launchpad-pqm/launchpad/devel/files/head:/lib/canonical/launchpad/mailnotification.py