The cause of this issue was nicely tracked down by Ryan Palmer on the Acquia forums: http://acquia.com/node/1011148

It turns out that if you have the Send module enabled alongside Acquia SPI, the expensive acquia_spi_send() function will get called all the time (rather than just once per 8 hours on cron like it is supposed to). The reason is that the way the function is named, it accidentally implements the Send module's hook_send()...

The attached patch simply renames the function to acquia_spi_send_profile_info() in order to avoid the problem.

CommentFileSizeAuthor
rename-acquia-spi-send.patch2.47 KBDavid_Rothstein

Comments

Ryan Palmer’s picture

Ryan Palmer commented
Status: Needs review » Reviewed & tested by the community

Patch looks good. Verified having both send and acquia_spi enabled no longer causes performance issues.

JacobSingh’s picture

JacobSingh commented

<rant> D8: We need to make a truly declarative plugin system. This "special" function name crap has gone one long enough. (intentionally not closing that tag).

pwolanin’s picture

pwolanin commented

Well - at the very least we could have __ or something else as the hook indicator to avoid random collisions like this.

pwolanin’s picture

pwolanin commented

see: http://drupal.org/node/548470

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Fixed

committed to HEAD and 6.x branches.

We will need to roll a new release for this bug fix before long, so let's see if there is anything else.

Status: Fixed » Closed (fixed)

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