I've found that meq honors an undocumented 'leasetime' option in hook_cron_queue_info() (which Drupal's drupal_cron_run does not).
It's a useful feature to have it in the hook_cron_queue_info(), but for third party modules or enhanced configurability I would like to add a 'mob_queue_QUEUE_leasetime' option.
The value and fallback now is:
hook_cron_queue_info['leasetime'] => 30
With the option would be:
hook_cron_queue_info['leasetime'] => variable_get('mob_queue_QUEUE_leasetime') => 30
This would not affect current behaviour for unconfigured queues and would not affect also queues using 'leasetime' option in hook_cron_queue_info().
Does it make sense to submit a patch?
Comments
Comment #2
wuinfo - Bill Wu CreditAttribution: wuinfo - Bill Wu commentedIt is nice to have this default value configurable. But it increases the complexity. Let's leave it for the developer rather than the site builder.
I think we can just leave as is for now. Thanks.