Skip to content

Commit

Permalink
Fix drush performance command.
Browse files Browse the repository at this point in the history
It seems our internal changes have made the invoke method no longer work. I believe the
results are still comparable

On 4.6 we recorded on T127133 between 308 & 333 imports per minute.

I just ran on staging now & got between 345 & 411 per minute. Notably the
345 was the first run & presumably some mysql caching kicked in after that
as subsequent runs were faster.

command is
drush profile_donation 1000 "4.7.31 with dao patch"

I did not see a speed difference with or without civicrm/civicrm-core#11615
(once mysql was 'warmed up') although there was a memory usage difference 77.2 MB used vs 83.97MB
without the destruct patch.

Change-Id: I13228690c7d225963442386e26355c47842ca631
  • Loading branch information
eileenmcnaughton authored and ejegg committed Feb 21, 2018
1 parent 6a3a27f commit 5c4688a
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ function drush_queue_performance_profile_donation($limit, $context) {

$now = strtotime('now');
foreach ($messages as $msg) {
module_invoke('queue2civicrm', 'import', $msg);
wmf_civicrm_contribution_message_import($msg);
}
$timeTaken = strtotime('now') - $now;
drush_print(t("Processed @limit messages in $timeTaken second(s)", array('@limit' => $limit)));
Expand Down

0 comments on commit 5c4688a

Please sign in to comment.