Problem/Motivation

Currently the cron time for the NodeRevisionDelete is 300 seconds!

#[QueueWorker(
    id: 'node_revision_delete',
    title: new TranslatableMarkup('Node revision delete'),
    cron: ['time' => 300]
)]

That feels like a very very long time, especially as \Drupal\Core\Cron::run() does 

    // Try to allocate enough time to run all the hook_cron implementations.
    Environment::setTimeLimit(240);

Proposed resolution

On the 2.1.x branch has performance improvements and queue processing improvements that make this unnecessary. Let's reduce this to 30 seconds.

Also the NodeRevisionDeleteRequeue has a completely unnecessary 300 second cron time too. That queue will be really quick to process so let's reduce it to 5 seconds.

Remaining tasks

User interface changes

None

API changes

None

Data model changes

None

Issue fork node_revision_delete-3584331

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

alexpott created an issue. See original summary.

alexpott opened merge request !65

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Reviewed & tested by the community

This is a simple one.

  • alexpott committed 46b06a16 on 2.1.x 
    task: #3584331 The cron time for the node revision delete queue is...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

alexpott closed merge request !65

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.