Page MenuHomePhabricator

Create jobs in job queue for sending email notification
Closed, ResolvedPublic


Here are the reasons why we should use job queue:

  • send notification is NOT urgent job. It could be done by the job runner.
  • for popular categories (which has a lot watchers), this email sending process could drag down performance significantly!

I need do some research on this... It will have more sub-tasks for this.

Related Objects

Event Timeline

I've added integration with Echo that could close this.

Change 386007 had a related patch set uploaded (by MarkAHershberger; owner: MarkAHershberger):
[mediawiki/extensions/CategoryWatch@master] add Echo support

Change 386007 merged by Seanchenldap:
[mediawiki/extensions/CategoryWatch@master] add Echo support

@Seanchen: Hi, the patch in Gerrit has been merged. Can this task be resolved (via Add Action...Change Status in the dropdown menu), or is there more to do in this task? Asking as you are set as task assignee. Thanks in advance!

Aklapper reassigned this task from Seanchen to MarkAHershberger.

@Seanchen: No reply, hence assuming this is resolved.