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

amontero created an issue. See original summary.

wuinfo - Bill Wu’s picture

Status: Active » Closed (works as designed)

It 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.