Make sure job queue partitions use the null aggregator
authorAaron Schulz <aschulz@wikimedia.org>
Thu, 4 Jun 2015 23:53:56 +0000 (16:53 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Fri, 5 Jun 2015 00:02:45 +0000 (00:02 +0000)
* Fixes problem from 37042262e32e
* The main class already handles the calls.
* Partitions call notifyQueueNonEmpty() on pop() even
  when the other partitions have jobs, so it is also wrong
  in addition to redundant.

Bug: T101427
Change-Id: Ic3235e1f2038053fabf92f97c663479a21d75317

includes/jobqueue/JobQueueFederated.php

index ecbb031..109ca01 100644 (file)
@@ -93,6 +93,8 @@ class JobQueueFederated extends JobQueue {
                ) {
                        unset( $baseConfig[$o] ); // partition queue doesn't care about this
                }
+               // The class handles all aggregator calls already
+               unset( $baseConfig['aggregator'] );
                // Get the partition queue objects
                foreach ( $partitionMap as $partition => $w ) {
                        if ( !isset( $params['configByPartition'][$partition] ) ) {