Uploaded image for project: 'CiviCRM'
  1. CiviCRM
  2. CRM-17631

Mailings: process jobs by scheduled date instead of mailing ID

    Details

    • Type: Improvement
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 4.7
    • Fix Version/s: 4.7
    • Component/s: None
    • Labels:
    • Documentation Required?:
      None
    • Funding Source:
      Contributed Code

      Description

      Currently mailing jobs are processed in the order of their creation (by ID number). It would be better if they were processed in order of their scheduled date.

      Use case:
      Two mailings scheduled: 16:00, 16:01.
      The 16:01 mailing was created first, so its mailing jobs are processed first. Ideally the script would begin sending the 16:00 mailing first - since it was scheduled first.

      Should be a really quick change here:
      https://github.com/civicrm/civicrm-core/blob/8b60bc736eb6782b7ebe48c24f2dbf57cced493a/CRM/Mailing/BAO/MailingJob.php#L126
      to:
      ORDER BY j.scheduled_date ASC, j.mailing_id

      I'll make a PR when I get the chance. Just noting this for now.

        Attachments

          Activity

            People

            • Assignee:
              yashodha Yashodha Chaku
              Reporter:
              john John K.
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: