In services_client_error_cron() the services errors are being deleted one by one. In case there is reasonable number of these items the deletion process becomes inefficient.

CommentFileSizeAuthor
#2 services_client_2839348.patch1.08 KBblueminds
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

blueminds created an issue. See original summary.

blueminds’s picture

blueminds CreditAttribution: blueminds commented
FileSize
services_client_2839348.patch1.08 KB
milsyobtaf’s picture

milsyobtaf CreditAttribution: milsyobtaf at Acquia commented
Status: Active » Reviewed & tested by the community

This looks good. Down the road it might be worthwhile to refactor services_client_error_delete() to accept multiple $eids, to DRY up the code by only defining the DB queries in one place, but this is a great first step.

  • 6d47d6e committed on 7.x-2.x 
    Issue #2839348 by blueminds: Delete services client errors in one query
blueminds’s picture

blueminds CreditAttribution: blueminds commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.