With the new queue we don't need "delivery_batch_count" anymore
This commit is contained in:
parent
12659fc3a1
commit
69daaa61ba
|
@ -638,7 +638,7 @@ function notifier_run(&$argv, &$argc){
|
|||
|
||||
// This controls the number of deliveries to execute with each separate delivery process.
|
||||
// By default we'll perform one delivery per process. Assuming a hostile shared hosting
|
||||
// provider, this provides the greatest chance of deliveries if processes start getting
|
||||
// provider, this provides the greatest chance of deliveries if processes start getting
|
||||
// killed. We can also space them out with the delivery_interval to also help avoid them
|
||||
// getting whacked.
|
||||
|
||||
|
@ -646,8 +646,10 @@ function notifier_run(&$argv, &$argc){
|
|||
// together into a single process. This will reduce the overall number of processes
|
||||
// spawned for each delivery, but they will run longer.
|
||||
|
||||
// When using the workerqueue, we don't need this functionality.
|
||||
|
||||
$deliveries_per_process = intval(get_config('system','delivery_batch_count'));
|
||||
if($deliveries_per_process <= 0)
|
||||
if (($deliveries_per_process <= 0) OR get_config("system", "worker"))
|
||||
$deliveries_per_process = 1;
|
||||
|
||||
$this_batch = array();
|
||||
|
|
Loading…
Reference in a new issue