CRM-4002 CiviMailProcessor.php runs into race condition on catchall accounts with a lot of traffic

    Details

    • Type: Improvement
    • Status: Done/Fixed
    • Priority: Trivial
    • Resolution: Fixed/Completed
    • Affects Version/s: 2.1.2, 2.1.4, 2.1
    • Fix Version/s: 2.2.0
    • Component/s: CiviMail
    • Labels:
      None

      Description

      In 2.1 CiviMailProcessor.php tries to process all mails in the inbox. This might not be possible due to several reasons - for sure it would be time limit on script running time, or other limits (currently have a few thousands mails in the catchall, not sure what exactly breaks, maybe even memory).

      In any case much better approach would be to process it chunk by chunk.

      See discussion http://forum.civicrm.org/index.php/topic,5997.0.html

        Attachments

          Activity

          [CRM-4002] CiviMailProcessor.php runs into race condition on catchall accounts with a lot of traffic
          Piotr Szotkowski added a comment -

          This is mostly done in r19515 and r19516, but (surprisingly) raises an exception when finishing; will finish my investigations on Monday and (hopefully) resolve.

          Piotr Szotkowski added a comment -

          Fixed in various commits (r19515, r19516, r19517, r19526, r19529, r19541, r19543).

          Needs verification - it works for me with Gmail's IMAP and POP, but ideally we'd like to test it on other IMAP and POP servers as well.

          Kurund Jalmi added a comment -

          Batch move to verification

          Yashodha Chaku added a comment -

          assigning for 3.0 QA

          Neha Kulkarni added a comment -

          verified in rev 23536

            People

            • Assignee:
              Neha Kulkarni
              Reporter:
              goran gugic

              Dates

              • Created:
                Updated:
                Resolved: